Skip to content

Commit

Permalink
Re-enable -Wshadow by default.
Browse files Browse the repository at this point in the history
Bug: 1344231
Change-Id: If0c553615902a32f7dad373ab194ff5c7a883335
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3939995
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1056607}
  • Loading branch information
pkasting authored and Chromium LUCI CQ committed Oct 8, 2022
1 parent 7fb3160 commit 03048f7
Showing 1 changed file with 1 addition and 13 deletions.
14 changes: 1 addition & 13 deletions net/third_party/quiche/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -30,23 +30,15 @@ import("//third_party/protobuf/proto_library.gni")

build_epoll_based_tools = is_linux || is_chromeos

# TODO(crbug.com/1344231): Set this to always true. At which point, this
# variable can go away and the conditionals below that depends on it can be
# simplified.
_no_shadow_is_internal = !is_chromeos_lacros

config("quiche_internal_config") {
cflags = []
if (is_clang) {
cflags += [
"-Wno-unused-private-field",
"-Wno-shadow",
"-Wno-sign-compare",
]
}

if (_no_shadow_is_internal) {
cflags += [ "-Wno-shadow" ]
}
}

config("quiche_config") {
Expand All @@ -57,10 +49,6 @@ config("quiche_config") {
"src/quiche/common/platform/default",
"src",
]

if (!_no_shadow_is_internal) {
cflags = [ "-Wno-shadow" ]
}
}

component("quiche") {
Expand Down

0 comments on commit 03048f7

Please sign in to comment.