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

Unified Diff: third_party/libxslt/libxslt.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
Index: third_party/libxslt/libxslt.gyp
diff --git a/third_party/libxslt/libxslt.gyp b/third_party/libxslt/libxslt.gyp
index dea6ae83f0e864198d053e6be7138d10ecf9bf55..55525190cb0550fc6791b7e8aba6ab0fe56448d8 100644
--- a/third_party/libxslt/libxslt.gyp
+++ b/third_party/libxslt/libxslt.gyp
@@ -106,19 +106,14 @@
'.',
],
},
+ 'variables': {
+ 'clang_warning_flags': [
+ # libxslt stores a char[3] in a `const unsigned char*`.
+ '-Wno-pointer-sign',
+ ],
+ },
'conditions': [
['OS!="win"', {'product_name': 'xslt'}],
- ['clang == 1', {
- 'xcode_settings': {
- 'WARNING_CFLAGS': [
- # libxslt stores a char[3] in a `const unsigned char*`.
- '-Wno-pointer-sign',
- ],
- },
- 'cflags': [
- '-Wno-pointer-sign',
- ],
- }],
],
}],
],

Powered by Google App Engine
This is Rietveld 408576698