diff options
author | Chih-Hung Hsieh | 2017-10-10 16:59:44 -0500 |
---|---|---|
committer | Chih-Hung Hsieh | 2017-10-12 13:48:12 -0500 |
commit | 36def26b34cb8d7a133d8eb9bdeea00767dffbb8 (patch) | |
tree | 8ad29bc4bb62a665c85f5e30d507c9f33ab8bc3f | |
parent | 486358728120226bf5b48cc1ec7e14744bdaff33 (diff) | |
download | external-libdrm-36def26b34cb8d7a133d8eb9bdeea00767dffbb8.tar.gz external-libdrm-36def26b34cb8d7a133d8eb9bdeea00767dffbb8.tar.xz external-libdrm-36def26b34cb8d7a133d8eb9bdeea00767dffbb8.zip |
Use -Werror in external/libdrm
* Suppress unused-parameter, unused-variable, pointer-arith,
format, gnu-variable-sized-type-not-at-end warnings.
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I74fdde60e697d77b90644ea9e94005dc6d14a9a6
-rw-r--r-- | Android.common.mk | 6 | ||||
-rw-r--r-- | Android.mk | 1 | ||||
-rw-r--r-- | rockchip/Android.mk | 3 | ||||
-rw-r--r-- | tegra/Android.mk | 3 | ||||
-rw-r--r-- | tests/planetest/Android.mk | 4 |
5 files changed, 15 insertions, 2 deletions
diff --git a/Android.common.mk b/Android.common.mk index 46370815..1fe79035 100644 --- a/Android.common.mk +++ b/Android.common.mk | |||
@@ -5,7 +5,13 @@ LOCAL_CFLAGS += \ | |||
5 | -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1 | 5 | -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1 |
6 | 6 | ||
7 | LOCAL_CFLAGS += \ | 7 | LOCAL_CFLAGS += \ |
8 | -Wall \ | ||
9 | -Werror \ | ||
10 | -Wno-format \ | ||
11 | -Wno-gnu-variable-sized-type-not-at-end \ | ||
12 | -Wno-pointer-arith \ | ||
8 | -Wno-unused-parameter \ | 13 | -Wno-unused-parameter \ |
14 | -Wno-unused-variable \ | ||
9 | -Wno-missing-field-initializers | 15 | -Wno-missing-field-initializers |
10 | 16 | ||
11 | # Quiet down the build system and remove any .h files from the sources | 17 | # Quiet down the build system and remove any .h files from the sources |
@@ -32,6 +32,7 @@ LOCAL_PATH := $(call my-dir) | |||
32 | include $(LOCAL_PATH)/Makefile.sources | 32 | include $(LOCAL_PATH)/Makefile.sources |
33 | 33 | ||
34 | common_CFLAGS := \ | 34 | common_CFLAGS := \ |
35 | -Wall -Werror \ | ||
35 | -Wno-enum-conversion \ | 36 | -Wno-enum-conversion \ |
36 | -Wno-pointer-arith \ | 37 | -Wno-pointer-arith \ |
37 | -Wno-sign-compare \ | 38 | -Wno-sign-compare \ |
diff --git a/rockchip/Android.mk b/rockchip/Android.mk index 05d421e2..82b3b00a 100644 --- a/rockchip/Android.mk +++ b/rockchip/Android.mk | |||
@@ -8,7 +8,8 @@ LOCAL_SHARED_LIBRARIES := libdrm | |||
8 | LOCAL_SRC_FILES := rockchip_drm.c | 8 | LOCAL_SRC_FILES := rockchip_drm.c |
9 | 9 | ||
10 | LOCAL_CFLAGS := \ | 10 | LOCAL_CFLAGS := \ |
11 | -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1 | 11 | -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1 \ |
12 | -Wall -Werror | ||
12 | 13 | ||
13 | LOCAL_SHARED_LIBRARIES := \ | 14 | LOCAL_SHARED_LIBRARIES := \ |
14 | libdrm | 15 | libdrm |
diff --git a/tegra/Android.mk b/tegra/Android.mk index d2a1a59d..7300728c 100644 --- a/tegra/Android.mk +++ b/tegra/Android.mk | |||
@@ -8,7 +8,8 @@ LOCAL_SHARED_LIBRARIES := libdrm | |||
8 | LOCAL_SRC_FILES := tegra.c | 8 | LOCAL_SRC_FILES := tegra.c |
9 | 9 | ||
10 | LOCAL_CFLAGS := \ | 10 | LOCAL_CFLAGS := \ |
11 | -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1 | 11 | -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1 \ |
12 | -Wall -Werror -Wno-unused-variable | ||
12 | 13 | ||
13 | LOCAL_SHARED_LIBRARIES := \ | 14 | LOCAL_SHARED_LIBRARIES := \ |
14 | libdrm | 15 | libdrm |
diff --git a/tests/planetest/Android.mk b/tests/planetest/Android.mk index 3616e2b9..4a56d135 100644 --- a/tests/planetest/Android.mk +++ b/tests/planetest/Android.mk | |||
@@ -3,6 +3,8 @@ LOCAL_PATH := $(call my-dir) | |||
3 | include $(CLEAR_VARS) | 3 | include $(CLEAR_VARS) |
4 | include $(LOCAL_PATH)/Makefile.sources | 4 | include $(LOCAL_PATH)/Makefile.sources |
5 | 5 | ||
6 | LOCAL_CFLAGS := -Wall -Werror -Wno-unused-parameter -Wno-pointer-arith | ||
7 | |||
6 | LOCAL_SRC_FILES := $(filter-out %.h,$(PLANETEST_COMMON_FILES) $(PLANETEST_FILES)) | 8 | LOCAL_SRC_FILES := $(filter-out %.h,$(PLANETEST_COMMON_FILES) $(PLANETEST_FILES)) |
7 | 9 | ||
8 | LOCAL_MODULE := planetest | 10 | LOCAL_MODULE := planetest |
@@ -14,6 +16,8 @@ include $(BUILD_EXECUTABLE) | |||
14 | include $(CLEAR_VARS) | 16 | include $(CLEAR_VARS) |
15 | include $(LOCAL_PATH)/Makefile.sources | 17 | include $(LOCAL_PATH)/Makefile.sources |
16 | 18 | ||
19 | LOCAL_CFLAGS := -Wall -Werror -Wno-unused-parameter -Wno-pointer-arith | ||
20 | |||
17 | LOCAL_SRC_FILES := $(filter-out %.h,$(PLANETEST_COMMON_FILES) $(ATOMICTEST_FILES)) | 21 | LOCAL_SRC_FILES := $(filter-out %.h,$(PLANETEST_COMMON_FILES) $(ATOMICTEST_FILES)) |
18 | 22 | ||
19 | LOCAL_MODULE := atomictest | 23 | LOCAL_MODULE := atomictest |