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

Side by Side Diff: sql/BUILD.gn

Issue 929793006: Update existing uses of /wd4267 to use the GN config (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@x64
Patch Set: Created 5 years, 10 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 | « rlz/BUILD.gn ('k') | storage/browser/BUILD.gn » ('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("//testing/test.gni") 5 import("//testing/test.gni")
6 6
7 component("sql") { 7 component("sql") {
8 sources = [ 8 sources = [
9 "connection.cc", 9 "connection.cc",
10 "connection.h", 10 "connection.h",
11 "error_delegate_util.cc", 11 "error_delegate_util.cc",
12 "error_delegate_util.h", 12 "error_delegate_util.h",
13 "init_status.h", 13 "init_status.h",
14 "meta_table.cc", 14 "meta_table.cc",
15 "meta_table.h", 15 "meta_table.h",
16 "recovery.cc", 16 "recovery.cc",
17 "recovery.h", 17 "recovery.h",
18 "statement.cc", 18 "statement.cc",
19 "statement.h", 19 "statement.h",
20 "transaction.cc", 20 "transaction.cc",
21 "transaction.h", 21 "transaction.h",
22 ] 22 ]
23 23
24 # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
25 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
26
24 defines = [ "SQL_IMPLEMENTATION" ] 27 defines = [ "SQL_IMPLEMENTATION" ]
25 28
26 if (is_win) {
27 cflags = [ "/wd4267" ] # size_t to int.
28 }
29
30 deps = [ 29 deps = [
31 "//base", 30 "//base",
32 "//third_party/sqlite", 31 "//third_party/sqlite",
33 "//base/third_party/dynamic_annotations", 32 "//base/third_party/dynamic_annotations",
34 ] 33 ]
35 } 34 }
36 35
37 source_set("test_support") { 36 source_set("test_support") {
38 testonly = true 37 testonly = true
39 sources = [ 38 sources = [
(...skipping 21 matching lines...) Expand all
61 "sqlite_features_unittest.cc", 60 "sqlite_features_unittest.cc",
62 "statement_unittest.cc", 61 "statement_unittest.cc",
63 "test/paths.cc", 62 "test/paths.cc",
64 "test/paths.h", 63 "test/paths.h",
65 "test/run_all_unittests.cc", 64 "test/run_all_unittests.cc",
66 "test/sql_test_suite.cc", 65 "test/sql_test_suite.cc",
67 "test/sql_test_suite.h", 66 "test/sql_test_suite.h",
68 "transaction_unittest.cc", 67 "transaction_unittest.cc",
69 ] 68 ]
70 69
71 if (is_win) { 70 # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
72 cflags = [ "/wd4267" ] # size_t -> int 71 configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
73 }
74 72
75 deps = [ 73 deps = [
76 ":sql", 74 ":sql",
77 ":test_support", 75 ":test_support",
78 "//base/allocator", 76 "//base/allocator",
79 "//base/test:test_support", 77 "//base/test:test_support",
80 "//testing/gtest", 78 "//testing/gtest",
81 "//third_party/sqlite", 79 "//third_party/sqlite",
82 ] 80 ]
83 81
(...skipping 10 matching lines...) Expand all
94 #'target_name': 'sql_unittests_apk', 92 #'target_name': 'sql_unittests_apk',
95 #'type': 'none', 93 #'type': 'none',
96 #'dependencies': [ 94 #'dependencies': [
97 # 'sql_unittests', 95 # 'sql_unittests',
98 #], 96 #],
99 #'variables': { 97 #'variables': {
100 # 'test_suite_name': 'sql_unittests', 98 # 'test_suite_name': 'sql_unittests',
101 #}, 99 #},
102 #'includes': [ '../build/apk_test.gypi' ], 100 #'includes': [ '../build/apk_test.gypi' ],
103 } 101 }
OLDNEW
« no previous file with comments | « rlz/BUILD.gn ('k') | storage/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698