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

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

Issue 1557733002: Prepare for -Wall for third-party code, -Wextra for chromium_code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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
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 if (current_toolchain == host_toolchain) { 5 if (current_toolchain == host_toolchain) {
6 config("android_protoc_warnings") {
7 cflags = [
8 "-Wno-null-conversion",
9 "-Wno-tautological-undefined-compare",
10 "-Wno-unused-function",
11 "-Wno-unused-local-typedef",
12 ]
13 }
14
6 # GYP: //third_party/android_protobuf/android_protobuf.gyp:android_protoc 15 # GYP: //third_party/android_protobuf/android_protobuf.gyp:android_protoc
7 executable("android_protoc") { 16 executable("android_protoc") {
8 configs -= [ "//build/config/compiler:chromium_code" ] 17 configs -= [ "//build/config/compiler:chromium_code" ]
9 configs += [ "//build/config/compiler:no_chromium_code" ] 18 configs += [
19 "//build/config/compiler:no_chromium_code",
20
21 # Must be after no_chromium_code so warning flags are ordered correctly.
22 ":android_protoc_warnings",
23 ]
10 24
11 deps = [ 25 deps = [
12 "//build/config/sanitizers:deps", 26 "//build/config/sanitizers:deps",
13 "//third_party/zlib", 27 "//third_party/zlib",
14 ] 28 ]
15 29
16 sources = [ 30 sources = [
17 "src/src/google/protobuf/compiler/code_generator.cc", 31 "src/src/google/protobuf/compiler/code_generator.cc",
18 "src/src/google/protobuf/compiler/command_line_interface.cc", 32 "src/src/google/protobuf/compiler/command_line_interface.cc",
19 "src/src/google/protobuf/compiler/cpp/cpp_enum.cc", 33 "src/src/google/protobuf/compiler/cpp/cpp_enum.cc",
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 "src/src/google/protobuf/unknown_field_set.cc", 121 "src/src/google/protobuf/unknown_field_set.cc",
108 "src/src/google/protobuf/wire_format.cc", 122 "src/src/google/protobuf/wire_format.cc",
109 "src/src/google/protobuf/wire_format_lite.cc", 123 "src/src/google/protobuf/wire_format_lite.cc",
110 ] 124 ]
111 125
112 include_dirs = [ 126 include_dirs = [
113 "src/android", 127 "src/android",
114 "src/src", 128 "src/src",
115 ] 129 ]
116 130
117 if (is_clang) {
118 cflags = [
119 "-Wno-null-conversion",
120 "-Wno-tautological-undefined-compare",
121 ]
122 }
123 defines = [ 131 defines = [
124 # This macro must be defined to suppress the use 132 # This macro must be defined to suppress the use
125 # of dynamic_cast<>, which requires RTTI. 133 # of dynamic_cast<>, which requires RTTI.
126 "GOOGLE_PROTOBUF_NO_RTTI", 134 "GOOGLE_PROTOBUF_NO_RTTI",
127 "GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER", 135 "GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER",
128 ] 136 ]
129 } 137 }
130 } 138 }
131 139
132 if (is_android) { 140 if (is_android) {
(...skipping 10 matching lines...) Expand all
143 "src/java/src/main/java/com/google/protobuf/nano/FieldData.java", 151 "src/java/src/main/java/com/google/protobuf/nano/FieldData.java",
144 "src/java/src/main/java/com/google/protobuf/nano/InternalNano.java", 152 "src/java/src/main/java/com/google/protobuf/nano/InternalNano.java",
145 "src/java/src/main/java/com/google/protobuf/nano/InvalidProtocolBufferNano Exception.java", 153 "src/java/src/main/java/com/google/protobuf/nano/InvalidProtocolBufferNano Exception.java",
146 "src/java/src/main/java/com/google/protobuf/nano/MessageNano.java", 154 "src/java/src/main/java/com/google/protobuf/nano/MessageNano.java",
147 "src/java/src/main/java/com/google/protobuf/nano/MessageNanoPrinter.java", 155 "src/java/src/main/java/com/google/protobuf/nano/MessageNanoPrinter.java",
148 "src/java/src/main/java/com/google/protobuf/nano/UnknownFieldData.java", 156 "src/java/src/main/java/com/google/protobuf/nano/UnknownFieldData.java",
149 "src/java/src/main/java/com/google/protobuf/nano/WireFormatNano.java", 157 "src/java/src/main/java/com/google/protobuf/nano/WireFormatNano.java",
150 ] 158 ]
151 } 159 }
152 } 160 }
OLDNEW
« no previous file with comments | « third_party/android_crazy_linker/src/src/crazy_linker_leb128.h ('k') | third_party/android_protobuf/android_protobuf.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698