Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1767)

Unified Diff: build/common.gypi

Issue 11746004: Remove Android only warnings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Updated comment Created 7 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 7d1eca815bd6a44cb9af223ff56e39f6ec75abaf..58dcf1268c402791db51005595975f53ea54ee35 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -3053,20 +3053,14 @@
# for third party code. This works because cflags are added
# before defines.
'-U_FORTIFY_SOURCE',
- # Chromium builds its own (non-third-party) code with
- # -Werror to make all warnings into errors. However, Android
- # enables warnings that Chromium doesn't, so some of these
- # extra warnings trip and break things.
- # For now, we leave these warnings enabled but prevent them
- # from being treated as errors.
- #
+ # Disable warnings enabled by the Android build system
joth 2013/01/03 18:50:06 nit: flesh this out to: Disable any additional war
Kristian Monsen 2013/01/03 20:10:18 Done.
# Things that are part of -Wextra:
- '-Wno-error=extra', # Enabled by -Wextra, but no specific flag
- '-Wno-error=ignored-qualifiers',
- '-Wno-error=type-limits',
+ '-Wno-extra', # Enabled by -Wextra, but no specific flag
+ '-Wno-ignored-qualifiers',
+ '-Wno-type-limits',
# Other things unrelated to -Wextra:
- '-Wno-error=non-virtual-dtor',
- '-Wno-error=sign-promo',
+ '-Wno-non-virtual-dtor',
+ '-Wno-sign-promo',
],
'cflags_cc': [
# Disabling c++0x-compat should be handled in WebKit, but
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698