clang/win: Build chromium code without -Wno-incompatible-pointer-types.

third_party/libwebp/libwebp.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'targets': [
     {
       'target_name': 'libwebp_dec',
       'type': 'static_library',
       'dependencies' : [
@@ skipping 15 matching lines @@
         'dec/webp.c',
       ],
     },
     {
       'target_name': 'libwebp_demux',
       'type': 'static_library',
       'include_dirs': ['.'],
       'sources': [
         'demux/demux.c',
       ],
+      'dependencies' : [
+        'libwebp_utils',

[fbarchard, 2015/07/10 19:26:42]

pulling in utilities as a dependency seems like overkill.  would be better to
apply the option to the target, or rely on common.gpyi.

[Sam McNally, 2015/07/13 08:00:25]

demux.c includes utils/utils.h so it should depend on the target containing it.
libwebp_utils probably should list its headers in sources.

+      ],
     },
     {
       'target_name': 'libwebp_dsp',
       'type': 'static_library',
       'include_dirs': ['.'],
       'sources': [
         'dsp/alpha_processing.c',
         'dsp/alpha_processing_sse2.c',
         'dsp/cpu.c',
         'dsp/dec.c',
         'dsp/dec_clip_tables.c',
         'dsp/dec_mips32.c',
         'dsp/dec_sse2.c',
         'dsp/enc.c',
         'dsp/enc_avx2.c',
         'dsp/enc_mips32.c',
         'dsp/enc_sse2.c',
         'dsp/lossless.c',
         'dsp/lossless_mips32.c',
         'dsp/lossless_sse2.c',
         'dsp/upsampling.c',
         'dsp/upsampling_sse2.c',
         'dsp/yuv.c',
         'dsp/yuv_mips32.c',
         'dsp/yuv_sse2.c',
       ],
+      'dependencies' : [
+        'libwebp_utils',
+      ],
       'conditions': [
         ['OS == "android"', {
           'dependencies': [ '../../build/android/ndk.gyp:cpu_features' ],
         }],
         ['order_profiling != 0', {
           'target_conditions' : [
             ['_toolset=="target"', {
               'cflags!': [ '-finstrument-functions' ],
             }],
           ],
@@ skipping 62 matching lines @@
         'enc/picture_psnr.c',
         'enc/picture_rescale.c',
         'enc/picture_tools.c',
         'enc/quant.c',
         'enc/syntax.c',
         'enc/token.c',
         'enc/tree.c',
         'enc/vp8l.c',
         'enc/webpenc.c',
       ],
+      'dependencies' : [
+        'libwebp_utils',
+      ],
     },
     {
       'target_name': 'libwebp_utils',
       'type': 'static_library',
       'include_dirs': ['.'],
       'sources': [
         'utils/bit_reader.c',
         'utils/bit_writer.c',
         'utils/color_cache.c',
         'utils/filters.c',
         'utils/huffman.c',
         'utils/huffman_encode.c',
         'utils/quant_levels.c',
         'utils/quant_levels_dec.c',
         'utils/random.c',
         'utils/rescaler.c',
         'utils/thread.c',
         'utils/utils.c',
       ],
+      'variables': {
+        'clang_warning_flags': [
+          # See https://code.google.com/p/webp/issues/detail?id=253.
+          '-Wno-incompatible-pointer-types',
+        ]
+      },
+      'direct_dependent_settings': {
+        'variables': {
+          'clang_warning_flags': [
+            # See https://code.google.com/p/webp/issues/detail?id=253.
+            '-Wno-incompatible-pointer-types',
+          ]
+        },
+      },
     },
     {
       'target_name': 'libwebp',
       'type': 'none',
       'dependencies' : [
         'libwebp_dec',
         'libwebp_demux',
         'libwebp_dsp',
         'libwebp_dsp_neon',
         'libwebp_enc',
         'libwebp_utils',
       ],
       'direct_dependent_settings': {
         'include_dirs': ['.'],
       },
       'conditions': [
         ['OS!="win"', {'product_name': 'webp'}],
       ],
     },
   ],
 }