diff options
author | thakis <thakis@chromium.org> | 2016-01-08 11:37:43 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-08 19:38:43 +0000 |
commit | 6fc6aa0987377a5d5633103de9c153fcccbf2ed8 (patch) | |
tree | cad74b3b73f2791037acedda2d4ac0d65350a94b /build/common.gypi | |
parent | 4bc8e916048df9c2559c3c479054a2cd4e632aa7 (diff) | |
download | chromium_src-6fc6aa0987377a5d5633103de9c153fcccbf2ed8.zip chromium_src-6fc6aa0987377a5d5633103de9c153fcccbf2ed8.tar.gz chromium_src-6fc6aa0987377a5d5633103de9c153fcccbf2ed8.tar.bz2 |
Enable warning on reserved user-defined literals everywhere but CrOS.
This was disabled because a dbus header was missing a few spaces.
The version of the dbus header in the sysroot has them though,
and we use the sysroot everywhere except in Chrome OS-on-Linux
builds these days.
The spaces were added here, almost 3 years ago by now:
http://cgit.freedesktop.org/dbus/dbus/commit/?h=dbus-1.4&id=51b88b4c7919487290c0862b013cd8e7cd2de34b
No behavior change.
BUG=263960, 573778
Review URL: https://codereview.chromium.org/1570193002
Cr-Commit-Position: refs/heads/master@{#368408}
Diffstat (limited to 'build/common.gypi')
-rw-r--r-- | build/common.gypi | 30 |
1 files changed, 24 insertions, 6 deletions
diff --git a/build/common.gypi b/build/common.gypi index e8c7b16..333a500 100644 --- a/build/common.gypi +++ b/build/common.gypi @@ -4315,10 +4315,6 @@ }]] }], ['clang==1', { - 'cflags': [ - # TODO(thakis): Remove, http://crbug.com/263960 - '-Wno-reserved-user-defined-literal', - ], 'cflags_cc': [ # gnu++11 instead of c++11 is needed because some code uses # typeof() (a GNU extension). @@ -4327,6 +4323,12 @@ '-std=gnu++11', ], }], + ['clang==1 and chromeos==1', { + 'cflags': [ + # TODO(thakis): Remove, http://crbug.com/263960 + '-Wno-reserved-user-defined-literal', + ], + }], ['clang==0 and host_clang==1', { 'target_conditions': [ ['_toolset=="host"', { @@ -4712,8 +4714,6 @@ '-std=gnu++11', # See comment for -Wno-c++11-narrowing. '-Wno-narrowing', - # TODO(thakis): Remove, http://crbug.com/263960 - '-Wno-literal-suffix', ], }], ], @@ -4725,6 +4725,24 @@ '-std=gnu++11', # See comment for -Wno-c++11-narrowing. '-Wno-narrowing', + ], + }], + ], + }], + ['gcc_version>=47 and clang==0 and chromeos==1', { + 'target_conditions': [ + ['_toolset=="target"', { + 'cflags_cc': [ + # TODO(thakis): Remove, http://crbug.com/263960 + '-Wno-literal-suffix', + ], + }], + ], + }], + ['host_gcc_version>=47 and clang==0 and host_clang==0 and chromeos==1', { + 'target_conditions': [ + ['_toolset=="host"', { + 'cflags_cc': [ # TODO(thakis): Remove, http://crbug.com/263960 '-Wno-literal-suffix', ], |