Fix some of compiler warnings. Also re-enabled redundant warnings that no-longer occur in code. Change-Id: I900cf1d732f96aaad40bee2b6da52fe43889331a Reviewed-on: https://weave-review.googlesource.com/2732 Reviewed-by: Alex Vakulenko <avakulenko@google.com>
diff --git a/Makefile b/Makefile index 9810dd0..70f7ee7 100644 --- a/Makefile +++ b/Makefile
@@ -28,13 +28,8 @@ -Wextra \ -Wformat=2 \ -Wl,--exclude-libs,ALL \ - -Wno-char-subscripts \ -Wno-missing-field-initializers \ - -Wno-unused-local-typedefs \ -Wno-unused-parameter \ - -Wpacked \ - -Wpointer-arith \ - -Wwrite-strings CFLAGS_Debug := \ -O0 \ @@ -56,7 +51,6 @@ CFLAGS := $(filter-out -Wl$(comma)--exclude-libs$(comma)ALL,$(CFLAGS)) CFLAGS += \ -fno-omit-frame-pointer \ - -Wno-deprecated-register \ -Wno-inconsistent-missing-override ifeq (Debug, $(BUILD_MODE)) CFLAGS += \
diff --git a/third_party/chromium/base/third_party/dmg_fp/dtoa.cc b/third_party/chromium/base/third_party/dmg_fp/dtoa.cc index 502c16c..c06219c 100644 --- a/third_party/chromium/base/third_party/dmg_fp/dtoa.cc +++ b/third_party/chromium/base/third_party/dmg_fp/dtoa.cc
@@ -59,6 +59,7 @@ * 4. Because of 3., we don't need a large table of powers of 10 * for ten-to-e (just some small tables, e.g. of 10^k * for 0 <= k <= 22). + * 5. Fixed -Wchar-subscripts warning in "if (!hexdig['0'])" */ /* @@ -1572,7 +1573,7 @@ CONST char *s; int c1, havedig, udx0, xshift; - if (!hexdig['0']) + if (!hexdig[0 + '0']) hexdig_init(); x[0] = x[1] = 0; havedig = xshift = 0;
diff --git a/third_party/chromium/base/third_party/dmg_fp/g_fmt.cc b/third_party/chromium/base/third_party/dmg_fp/g_fmt.cc index bfa358d..67c9f57 100644 --- a/third_party/chromium/base/third_party/dmg_fp/g_fmt.cc +++ b/third_party/chromium/base/third_party/dmg_fp/g_fmt.cc
@@ -27,10 +27,10 @@ namespace dmg_fp { char * -g_fmt(register char *b, double x) +g_fmt(char *b, double x) { - register int i, k; - register char *s; + int i, k; + char *s; int decpt, j, sign; char *b0, *s0, *se;