commit 5e5d1a65cbf855609e4d7c9c425615090a834704
author Ben Chan <benchan@chromium.org> Thu Feb 09 21:31:28 2017
committer chrome-bot <chrome-bot@chromium.org> Fri Feb 10 04:48:45 2017
tree 2336fb2f78b7b6fb9cea05114dea824225ee79e5
parent c9b5bb6cdf62abeb5b3a526c8d260656dff38f80

Turn compiler warnings into errors.

Gobi code still has lots of other compiler warnings if we turn on -Wall
and -Wextra. For now, we turn on -Werror to catch the default warnings
like -Wreturn-type.

BUG=chromium:690629
TEST=Build and run unit tests.

Change-Id: I2e5660e95e58442e779bfcea2bce0545752897a4
Reviewed-on: https://chromium-review.googlesource.com/440373
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

Makefile

diff --git a/Makefile b/Makefile
index 4e1e7c1..0f8b49a 100644
--- a/Makefile
+++ b/Makefile
@@ -4,7 +4,10 @@
 
 ROOT := $(DESTDIR)/opt/Qualcomm
 PREFIX := $(DESTDIR)
-CXXFLAGS += -fPIC
+CXXFLAGS += -fPIC -Werror
+# TODO(benchan): Gobi code has lots of other warnings, which should be cleaned
+# up eventually.
+CXXFLAGS += -Wno-switch
 
 sdk :
 	$(MAKE) -C GobiConnectionMgmt