From 11e31aedd3af9e95a860cfc1525badd4ffd7b56f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Boisnard?= Date: Tue, 6 Aug 2013 19:23:46 +0200 Subject: PFW: Update CFLAGS to add Wall, Werror, Wextra everywhere MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit BZ: 129048 Some Makefiles of the PFW projects are missing the regular CFLAGS -Wall, -Werror and -Wextra. This patch aims to update the CLFAGS of these Makefiles. Some source code had to be updated because warnings are now considered as errors. Change-Id: Icc91f7bdc649118f822411d9b4693776a84037e9 Signed-off-by: Frédéric Boisnard Reviewed-on: http://android.intel.com:8080/123977 Reviewed-by: Rocard, KevinX Reviewed-by: De Chivre, Renaud Tested-by: Dixon, CharlesX Reviewed-by: cactus Tested-by: cactus --- remote-processor/Android.mk | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'remote-processor') diff --git a/remote-processor/Android.mk b/remote-processor/Android.mk index d82d548..8facae7 100644 --- a/remote-processor/Android.mk +++ b/remote-processor/Android.mk @@ -16,6 +16,12 @@ common_src_files := \ common_module := libremote-processor common_module_tags := optional +common_cflags := \ + -Wall \ + -Werror \ + -Wextra \ + -Wno-unused-parameter + common_ldlibs := -lpthread ############################# # Target build @@ -27,6 +33,8 @@ LOCAL_SRC_FILES := $(common_src_files) LOCAL_MODULE := $(common_module) LOCAL_MODULE_TAGS := $(common_module_tags) +LOCAL_CFLAGS := $(common_cflags) + LOCAL_C_INCLUDES += \ $(call include-path-for, stlport) \ bionic/ @@ -44,6 +52,8 @@ include $(CLEAR_VARS) LOCAL_SRC_FILES := $(common_src_files) +LOCAL_CFLAGS := $(common_cflags) + LOCAL_MODULE := $(common_module)_host LOCAL_MODULE_TAGS := $(common_module_tags) -- cgit v1.1