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

Side by Side Diff: build/config/compiler/BUILD.gn

Issue 731373002: Enable MSVC warning for unused locals. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Build fix Created 6 years, 1 month 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 (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 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/android/config.gni") 5 import("//build/config/android/config.gni")
6 if (cpu_arch == "arm") { 6 if (cpu_arch == "arm") {
7 import("//build/config/arm.gni") 7 import("//build/config/arm.gni")
8 } 8 }
9 if (is_posix) { 9 if (is_posix) {
10 import("//build/config/gcc/gcc_version.gni") 10 import("//build/config/gcc/gcc_version.gni")
(...skipping 671 matching lines...) Expand 10 before | Expand all | Expand 10 after
682 # Chromium code compiles without exception support. We therefore have to 682 # Chromium code compiles without exception support. We therefore have to
683 # use setjmp/longjmp for e.g. JPEG decode error handling, which means we 683 # use setjmp/longjmp for e.g. JPEG decode error handling, which means we
684 # have to turn off this warning (and be careful about how object 684 # have to turn off this warning (and be careful about how object
685 # destruction happens in such cases). 685 # destruction happens in such cases).
686 "/wd4611", 686 "/wd4611",
687 687
688 688
689 # Warnings to evaluate and possibly fix/reenable later: 689 # Warnings to evaluate and possibly fix/reenable later:
690 690
691 "/wd4100", # Unreferenced formal function parameter. 691 "/wd4100", # Unreferenced formal function parameter.
692 "/wd4189", # A variable was declared and initialized but never used.
693 "/wd4244", # Conversion: possible loss of data. 692 "/wd4244", # Conversion: possible loss of data.
694 "/wd4481", # Nonstandard extension: override specifier. 693 "/wd4481", # Nonstandard extension: override specifier.
695 "/wd4505", # Unreferenced local function has been removed. 694 "/wd4505", # Unreferenced local function has been removed.
696 "/wd4510", # Default constructor could not be generated. 695 "/wd4510", # Default constructor could not be generated.
697 "/wd4512", # Assignment operator could not be generated. 696 "/wd4512", # Assignment operator could not be generated.
698 "/wd4610", # Class can never be instantiated, constructor required. 697 "/wd4610", # Class can never be instantiated, constructor required.
699 ] 698 ]
700 } else { 699 } else {
701 # Common GCC warning setup. 700 # Common GCC warning setup.
702 cflags = [ 701 cflags = [
(...skipping 265 matching lines...) Expand 10 before | Expand all | Expand 10 after
968 cflags += [ "-gsplit-dwarf" ] 967 cflags += [ "-gsplit-dwarf" ]
969 } 968 }
970 } 969 }
971 } 970 }
972 971
973 config("no_symbols") { 972 config("no_symbols") {
974 if (!is_win) { 973 if (!is_win) {
975 cflags = [ "-g0" ] 974 cflags = [ "-g0" ]
976 } 975 }
977 } 976 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698