Refactor skia_warnings_as_errors

gyp/jsoncpp.gyp

Index: gyp/jsoncpp.gyp
===================================================================
--- gyp/jsoncpp.gyp	(revision 8270)
+++ gyp/jsoncpp.gyp	(working copy)
@@ -47,29 +47,33 @@
         ],
       },
       'conditions': [
-        [ 'skia_os == "mac"', {
-          'xcode_settings': {
-            'OTHER_CPLUSPLUSFLAGS!': [
-              '-Werror',
-            ]
-          },
-        }],
-        [ 'skia_os == "win"', {
-          'msvs_settings': {
-            'VCCLCompilerTool': {
-              'WarnAsError': 'false',
-            },
-          },
-        }],
         [ 'skia_os == "android"', {
           'cflags!': [
             '-Wall',
-            '-Werror',
           ],
         }],
-        ['skia_os in ["linux", "freebsd", "openbsd", "solaris", "nacl"]', {
-          'cflags!': [
-            '-Werror',
+        # Turn off warnings as errors.

[djsollen, 2013/03/20 19:29:07]

What do you think of putting this in a gypi and calling something like.

'include': [ 'utils/disable_warnings_as_errors.gypi]

[borenet, 2013/03/20 20:02:50]

I like it! Done.

+        [ 'skia_warnings_as_errors', {
+          'conditions': [
+            [ 'skia_os == "mac" or skia_os == "ios"', {
+              'xcode_settings': {
+                'OTHER_CPLUSPLUSFLAGS!': [
+                  '-Werror',
+                ]
+              },
+            }],
+            [ 'skia_os == "win"', {
+              'msvs_settings': {
+                'VCCLCompilerTool': {
+                  'WarnAsError': 'false',
+                },
+              },
+            }],
+            ['skia_os in ["linux", "freebsd", "openbsd", "solaris", "nacl", "android"]', {
+              'cflags!': [
+                '-Werror',
+              ],
+            }],
           ],
         }],
       ],