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

Side by Side Diff: third_party/mesa/BUILD.gn

Issue 1236863009: clang/win: Enable -Wunused-local-typedef (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: unsetupset Created 5 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 unified diff | Download patch
« no previous file with comments | « third_party/harfbuzz-ng/harfbuzz.gyp ('k') | third_party/mesa/mesa.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/ui.gni") 5 import("//build/config/ui.gni")
6 6
7 config("mesa_headers_config") { 7 config("mesa_headers_config") {
8 include_dirs = [ "src/include" ] 8 include_dirs = [ "src/include" ]
9 if (use_x11) { 9 if (use_x11) {
10 defines = [ "MESA_EGL_NO_X11_HEADERS" ] 10 defines = [ "MESA_EGL_NO_X11_HEADERS" ]
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
118 # mesa_internal_config is prepended to the config lists. Flags that 118 # mesa_internal_config is prepended to the config lists. Flags that
119 # disable warnings need to be appended instead so that they show up 119 # disable warnings need to be appended instead so that they show up
120 # after flags like -Wall. (gn orders flags on a target before flags from 120 # after flags like -Wall. (gn orders flags on a target before flags from
121 # configs.) 121 # configs.)
122 config("mesa_internal_warnings") { 122 config("mesa_internal_warnings") {
123 cflags = [] 123 cflags = []
124 if (is_clang) { 124 if (is_clang) {
125 cflags += [ 125 cflags += [
126 "-Wno-tautological-constant-out-of-range-compare", 126 "-Wno-tautological-constant-out-of-range-compare",
127 "-Wno-mismatched-tags", # Fixed upstream. 127 "-Wno-mismatched-tags", # Fixed upstream.
128 # mesa's STATIC_ASSERT() macro expands to an ununused typedef.
129 "-Wno-unused-local-typedef",
128 ] 130 ]
129 } 131 }
130 if (is_win) { 132 if (is_win) {
131 # TODO(scottmg): http://crbug.com/143877 These should be removed if 133 # TODO(scottmg): http://crbug.com/143877 These should be removed if
132 # Mesa is ever rolled and the warnings are fixed. 134 # Mesa is ever rolled and the warnings are fixed.
133 cflags += [ 135 cflags += [
134 "/wd4005", # Macro defined twice. 136 "/wd4005", # Macro defined twice.
135 "/wd4065", # Switch statement contains 'default' but no 'case' labels. 137 "/wd4065", # Switch statement contains 'default' but no 'case' labels.
136 "/wd4090", # 'Operation' : different 'modifier' qualifiers 138 "/wd4090", # 'Operation' : different 'modifier' qualifiers
137 "/wd4099", # Type name struct-vs-class doesn't match. 139 "/wd4099", # Type name struct-vs-class doesn't match.
(...skipping 576 matching lines...) Expand 10 before | Expand all | Expand 10 after
714 "KEYWORD2=GLAPIENTRY", 716 "KEYWORD2=GLAPIENTRY",
715 ] 717 ]
716 } 718 }
717 } 719 }
718 } else { 720 } else {
719 # Placeholder to allow targets to unconditionally depend on this. 721 # Placeholder to allow targets to unconditionally depend on this.
720 group("osmesa") { 722 group("osmesa") {
721 } 723 }
722 } # !is_android 724 } # !is_android
723 # TODO(GYP) Android osmesa_in_lib_dir target. 725 # TODO(GYP) Android osmesa_in_lib_dir target.
OLDNEW
« no previous file with comments | « third_party/harfbuzz-ng/harfbuzz.gyp ('k') | third_party/mesa/mesa.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698