8 files changed, 8 insertions, 25 deletions
diff --git a/third_party/leveldatabase/BUILD.gn b/third_party/leveldatabase/BUILD.gn
index b3080d8..c29f0f8 100644
--- a/third_party/leveldatabase/BUILD.gn
+++ b/third_party/leveldatabase/BUILD.gn
@@ -113,10 +113,6 @@ static_library("leveldatabase") {
     "//third_party/re2",
     "//third_party/snappy",
   ]
-
-  if (is_win) {
-    cflags = [ "/wd4018" ]  # Signed/unsigned mismatch in comparison.
-  }
 }
 
 if (!is_android) {
diff --git a/third_party/libexif/BUILD.gn b/third_party/libexif/BUILD.gn
index 8eba113..5a464e6 100644
--- a/third_party/libexif/BUILD.gn
+++ b/third_party/libexif/BUILD.gn
@@ -60,7 +60,7 @@ if (!is_linux || is_chromeos) {
 
       ldflags = [ "/DEF:" + rebase_path("libexif.def") ]
 
-      cflags = [ "/wd4018" ]  # Signed/unsigned mismatch in comparison.
+      cflags = [ "/wd4018" ]  # size/unsigned mismatch
 
       # TODO(GYP): Handle /analyze switch, when it will be used in GN.
     } else if (is_linux) {
diff --git a/third_party/libxml/BUILD.gn b/third_party/libxml/BUILD.gn
index 58a5f1c..1f10dee 100644
--- a/third_party/libxml/BUILD.gn
+++ b/third_party/libxml/BUILD.gn
@@ -155,10 +155,7 @@ static_library("libxml") {
   ]
 
   if (is_win) {
-    cflags_c = [
-      "/wd4018",  # Signed/unsigned mismatch in comparison.
-      "/wd4101",  # Unreferenced local variable.
-    ]
+    cflags_c = [ "/wd4101" ]  # Unreferenced local variable.
   } else if (is_mac || is_ios || is_android) {
     # http://www.xmlsoft.org/threads.html says that this is required when using
     # libxml from several threads, which can possibly happen in chrome. On
diff --git a/third_party/mesa/BUILD.gn b/third_party/mesa/BUILD.gn
index 62f8611..96f5ec0 100644
--- a/third_party/mesa/BUILD.gn
+++ b/third_party/mesa/BUILD.gn
@@ -125,7 +125,6 @@ config("mesa_internal_warnings") {
     cflags += [
       "-Wno-tautological-constant-out-of-range-compare",
       "-Wno-mismatched-tags",  # Fixed upstream.
-
       # mesa's STATIC_ASSERT() macro expands to an ununused typedef.
       "-Wno-unused-local-typedef",
     ]
@@ -135,7 +134,6 @@ config("mesa_internal_warnings") {
     # Mesa is ever rolled and the warnings are fixed.
     cflags += [
       "/wd4005",  # Macro defined twice.
-      "/wd4018",  # Signed/unsigned mismatch in comparison.
       "/wd4065",  # Switch statement contains 'default' but no 'case' labels.
       "/wd4090",  # 'Operation' : different 'modifier' qualifiers
       "/wd4099",  # Type name struct-vs-class doesn't match.
diff --git a/third_party/protobuf/BUILD.gn b/third_party/protobuf/BUILD.gn
index 7d42a00d..0c57bd6 100644
--- a/third_party/protobuf/BUILD.gn
+++ b/third_party/protobuf/BUILD.gn
@@ -77,11 +77,11 @@ protobuf_lite_sources = [
 protobuf_lite_cflags = []
 if (is_win) {
   protobuf_lite_cflags = [
-    "/wd4018",  # Signed/unsigned mismatch in comparison.
-    "/wd4244",  # Implicit conversion, possible loss of data.
-    "/wd4355",  # 'this' used in base member initializer list.
-    "/wd4267",  # Size_t to int truncation.
-    "/wd4291",  # No matching operator delete for a placement new.
+    "/wd4018",  # signed/unsigned mismatch in comparison
+    "/wd4244",  # implicit conversion, possible loss of data
+    "/wd4355",  # 'this' used in base member initializer list
+    "/wd4267",  # size_t to int truncation
+    "/wd4291",  # no matching operator delete for a placement new
   ]
 }
 
diff --git a/third_party/qcms/BUILD.gn b/third_party/qcms/BUILD.gn
index a4b60d3..5e27d44 100644
--- a/third_party/qcms/BUILD.gn
+++ b/third_party/qcms/BUILD.gn
@@ -32,7 +32,6 @@ source_set("qcms") {
 
   if (is_win) {
     cflags = [
-      "/wd4018",  # Signed/unsigned mismatch in comparison.
       "/wd4056",  # Overflow in floating-point constant arithmetic (INFINITY).
       "/wd4756",  # Overflow in constant arithmetic (INFINITY).
     ]
diff --git a/third_party/re2/BUILD.gn b/third_party/re2/BUILD.gn
index ca1f7a2..2bc130a 100644
--- a/third_party/re2/BUILD.gn
+++ b/third_party/re2/BUILD.gn
@@ -67,10 +67,7 @@ static_library("re2") {
 
   if (is_win) {
     include_dirs = [ "mswin" ]
-    cflags = [
-      "/wd4018",  # Signed/unsigned mismatch in comparison.
-      "/wd4722",  # Destructor never terminates.
-    ]
+    cflags = [ "/wd4722" ]  # Destructor never terminates.
   } else {
     sources -= [ "mswin/stdint.h" ]
   }
diff --git a/third_party/snappy/BUILD.gn b/third_party/snappy/BUILD.gn
index e8de575..4af0e81 100644
--- a/third_party/snappy/BUILD.gn
+++ b/third_party/snappy/BUILD.gn
@@ -31,10 +31,6 @@ static_library("snappy") {
   configs += [ "//build/config/compiler:no_chromium_code" ]
   public_configs = [ ":snappy_config" ]
 
-  if (is_win) {
-    cflags = [ "/wd4018" ]  # Signed/unsigned mismatch in comparison.
-  }
-
   if (is_clang) {
     # snappy-stubs-internal.h unapologetically has: using namespace std
     # https://code.google.com/p/snappy/issues/detail?id=70