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

Unified Diff: third_party/zlib/zlib.gyp

Issue 437543007: Refactor how clang warning flags are set. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 5 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
« third_party/snappy/snappy.gyp ('K') | « third_party/sqlite/sqlite.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/zlib/zlib.gyp
diff --git a/third_party/zlib/zlib.gyp b/third_party/zlib/zlib.gyp
index 279a6e88c73b9befc7bfd9576b3805fb46039e03..aef41ac2bd969dedff9e8f44ec821e8dddc357fb 100644
--- a/third_party/zlib/zlib.gyp
+++ b/third_party/zlib/zlib.gyp
@@ -77,6 +77,12 @@
'.',
],
},
+ 'variables': {
+ 'clang_warning_flags': [
+ # zlib uses `if ((a == b))` for some reason.
+ '-Wno-parentheses-equality',
+ ],
+ },
'conditions': [
['OS!="win"', {
'sources!': [
@@ -94,17 +100,6 @@
'USE_FILE32API'
],
}],
- ['clang==1', {
- 'xcode_settings': {
- 'WARNING_CFLAGS': [
- # zlib uses `if ((a == b))` for some reason.
- '-Wno-parentheses-equality',
- ],
- },
- 'cflags': [
- '-Wno-parentheses-equality',
- ],
- }],
],
},
],
« third_party/snappy/snappy.gyp ('K') | « third_party/sqlite/sqlite.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698