ref: 1db36f4d3833f2e0d5e99486f6daca2059f10cea
parent: 132c0af3525dd4c26cc86ebe73bcf67d02cdc3ec
parent: 381eb3c7991dbbd0e5ae1f4c5f1fd9835d81328c
author: Johann Koenig <[email protected]>
date: Mon Dec 10 11:47:03 EST 2018
Merge "apply -Wextra to third_party/"
--- a/configure
+++ b/configure
@@ -614,10 +614,6 @@
check_add_cflags -Wimplicit-function-declaration
check_add_cflags -Wuninitialized
check_add_cflags -Wunused
- # -Wextra has some tricky cases. Rather than fix them all now, get the
- # flag for as many files as possible and fix the remaining issues
- # piecemeal.
- # https://bugs.chromium.org/p/webm/issues/detail?id=1069
check_add_cflags -Wextra
# check_add_cflags also adds to cxxflags. gtest does not do well with
# these flags so add them explicitly to CFLAGS only.
--- a/libs.mk
+++ b/libs.mk
@@ -112,10 +112,8 @@
CODEC_DOC_SECTIONS += decoder
endif
-# Suppress -Wextra warnings in third party code.
-$(BUILD_PFX)third_party/googletest/%.cc.o: CXXFLAGS += -Wno-missing-field-initializers
# Suppress -Wextra warnings in first party code pending investigation.
-# https://bugs.chromium.org/p/webm/issues/detail?id=1069
+# https://bugs.chromium.org/p/webm/issues/detail?id=1246
$(BUILD_PFX)vp8/encoder/onyx_if.c.o: CFLAGS += -Wno-unknown-warning-option -Wno-clobbered
$(BUILD_PFX)vp8/decoder/onyxd_if.c.o: CFLAGS += -Wno-unknown-warning-option -Wno-clobbered