From 373d3c7257fa815d0b9ee8f16874470a6002042e Mon Sep 17 00:00:00 2001 From: Chih-Hung Hsieh Date: Wed, 18 Oct 2017 16:28:14 -0700 Subject: Use -Werror in system/extras * Remove unused variables. * Fix redefined macro warnings. * Fix sign-compare warnings. * Fix 'return false' in main(). * Keep existing warnings to fix later. Bug: 66996870 Test: build with WITH_TIDY=1 Change-Id: Ib92ef5297693595fb84ed4f8e61665bda4cee312 --- ext4_utils/Android.mk | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'ext4_utils') diff --git a/ext4_utils/Android.mk b/ext4_utils/Android.mk index a80e473d..6f26dd64 100644 --- a/ext4_utils/Android.mk +++ b/ext4_utils/Android.mk @@ -21,6 +21,7 @@ LOCAL_SHARED_LIBRARIES_darwin += libselinux LOCAL_SHARED_LIBRARIES_linux += libselinux LOCAL_CFLAGS_darwin := -DHOST LOCAL_CFLAGS_linux := -DHOST +LOCAL_CFLAGS += -Wall -Werror include $(BUILD_HOST_EXECUTABLE) @@ -30,6 +31,7 @@ LOCAL_MODULE := blk_alloc_to_base_fs LOCAL_SHARED_LIBRARIES := libcutils LOCAL_CFLAGS_darwin := -DHOST LOCAL_CFLAGS_linux := -DHOST +LOCAL_CFLAGS += -Wall -Werror include $(BUILD_HOST_EXECUTABLE) # @@ -49,7 +51,7 @@ LOCAL_SHARED_LIBRARIES := \ libext4_utils \ libselinux \ libz -LOCAL_CFLAGS := -DREAL_UUID +LOCAL_CFLAGS := -DREAL_UUID -Wall -Werror include $(BUILD_EXECUTABLE) @@ -57,11 +59,13 @@ include $(CLEAR_VARS) LOCAL_SRC_FILES := setup_fs.c LOCAL_MODULE := setup_fs LOCAL_SHARED_LIBRARIES += libcutils +LOCAL_CFLAGS := -Wall -Werror include $(BUILD_EXECUTABLE) include $(CLEAR_VARS) LOCAL_SRC_FILES := ext4fixup_main.c +LOCAL_CFLAGS := -Wall -Werror LOCAL_MODULE := ext4fixup LOCAL_SHARED_LIBRARIES += \ libext4_utils \ @@ -72,6 +76,7 @@ include $(BUILD_EXECUTABLE) include $(CLEAR_VARS) LOCAL_SRC_FILES := ext4fixup_main.c +LOCAL_CFLAGS := -Wall -Werror LOCAL_MODULE := ext4fixup LOCAL_STATIC_LIBRARIES += \ libext4_utils \ -- cgit v1.2.3