Remove duplicated or non effective build flags.
-Winconsistent-missing-override is part of -Wall so there is no need
to explicitly set it.
-Wthread-safety and -Wimplicit-fallthrough are set by default by
Chromium's toolchain and there is no need to duplicated it in WebRTC
GN files:
gn desc out/Debug/ //rtc_base cflags | grep "Wthread-safety"
-Wthread-safety
-Wthread-safety
gn desc out/Debug/ //rtc_base cflags | grep "implicit"
-Wimplicit-fallthrough
-Wimplicit-fallthrough
Bug: None
Change-Id: Ie5104f7c6d508c7b45788420bf111a17b8b10939
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/226868
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#34549}
diff --git a/BUILD.gn b/BUILD.gn
index f2713f6..4afbdc7 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -344,9 +344,6 @@
if (is_clang) {
cflags += [
"-Wc++11-narrowing",
- "-Wimplicit-fallthrough",
- "-Wthread-safety",
- "-Winconsistent-missing-override",
"-Wundef",
]
