Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

am 26627d70: am 4d6cb77a: Merge "Tighten up compiler flags for aidl"

Browse Source 
* commit '26627d7011e8fbd66f0cabac5c3cb2a66c133d08':
  Tighten up compiler flags for aidl
This commit is contained in:
Christopher Wiley
2015-09-01 16:51:12 +00:00
committed by Android Git Automerger
parent 83533a235c 26627d7011
commit 5fa874a30b
2 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
tools/aidl/Android.mk
View File

@@ -8,6 +8,17 @@ ifeq ($(TARGET_BUILD_APPS)$(filter true,$(TARGET_BUILD_PDK)),)
LOCAL_PATH:= $(call my-dir) LOCAL_PATH:= $(call my-dir)
include $(CLEAR_VARS) include $(CLEAR_VARS)
LOCAL_CLANG_CFLAGS := -Wall -Werror
# Tragically, the code is riddled with unused parameters.
LOCAL_CLANG_CFLAGS += -Wno-unused-parameter
# yacc dumps a lot of code *just in case*.
LOCAL_CLANG_CFLAGS += -Wno-unused-function
LOCAL_CLANG_CFLAGS += -Wno-unneeded-internal-declaration
# yacc is a tool from a more civilized age.
LOCAL_CLANG_CFLAGS += -Wno-deprecated-register
# yacc also has a habit of using char* over const char*.
LOCAL_CLANG_CFLAGS += -Wno-writable-strings
LOCAL_SRC_FILES := \ LOCAL_SRC_FILES := \
AST.cpp \ AST.cpp \
Type.cpp \ Type.cpp \
@@ -22,7 +33,6 @@ LOCAL_SRC_FILES := \
options.cpp \ options.cpp \
search_path.cpp \ search_path.cpp \
LOCAL_CFLAGS := -g
LOCAL_MODULE := aidl LOCAL_MODULE := aidl
LOCAL_MODULE_HOST_OS := darwin linux windows LOCAL_MODULE_HOST_OS := darwin linux windows

5
tools/aidl/aidl.cpp
View File

@@ -162,11 +162,6 @@ import_import_parsed(buffer_type* statement)
{ {
} }
static ParserCallbacks g_importCallbacks = {
&main_document_parsed,
&import_import_parsed
};
// ========================================================== // ==========================================================
static int static int
check_filename(const char* filename, const char* package, buffer_type* name) check_filename(const char* filename, const char* package, buffer_type* name)