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-process/Android.mk | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'remote-process') diff --git a/remote-process/Android.mk b/remote-process/Android.mk index cc842f2..7cf3ba7 100644 --- a/remote-process/Android.mk +++ b/remote-process/Android.mk @@ -9,6 +9,11 @@ common_src_files := \ common_module := remote-process common_module_tags := optional +common_cflags := \ + -Wall \ + -Werror \ + -Wextra \ + -Wno-unused-parameter common_c_includes := \ $(LOCAL_PATH)/../remote-processor/ @@ -25,6 +30,8 @@ LOCAL_SRC_FILES := $(common_src_files) LOCAL_MODULE := $(common_module) LOCAL_MODULE_TAGS := $(common_module_tags) +LOCAL_CFLAGS := $(common_cflags) + LOCAL_C_INCLUDES += \ $(common_c_includes) \ $(call include-path-for, stlport) \ @@ -45,6 +52,8 @@ LOCAL_SRC_FILES := $(common_src_files) LOCAL_MODULE := $(common_module)_host LOCAL_MODULE_TAGS := $(common_module_tags) +LOCAL_CFLAGS := $(common_cflags) + LOCAL_C_INCLUDES += \ $(common_c_includes) -- cgit v1.1