Commit e08aa0a1 authored by Nico Weber's avatar Nico Weber

Enable -Wextra-semi in chromecast builds.

This seems to work after https://chromium-review.googlesource.com/c/chromium/src/+/1483332

Bug: 926235
Change-Id: I5a73ea6bf72a095be9222599b35b9544cfca3c73
Reviewed-on: https://chromium-review.googlesource.com/c/1486714
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: default avatarSergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#635518}
parent c6394e21
...@@ -1590,8 +1590,7 @@ config("chromium_code") { ...@@ -1590,8 +1590,7 @@ config("chromium_code") {
# TODO(thakis): Enable this for more platforms, https://crbug.com/926235 # TODO(thakis): Enable this for more platforms, https://crbug.com/926235
has_dchecks = is_debug || dcheck_always_on has_dchecks = is_debug || dcheck_always_on
if (has_dchecks && if (has_dchecks && ((is_linux && !use_libfuzzer) || is_mac)) {
((is_linux && !is_chromecast && !use_libfuzzer) || is_mac)) {
cflags += [ "-Wextra-semi" ] cflags += [ "-Wextra-semi" ]
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment