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

* commit '4d6cb77ad92c9acadb5afc5d80be38dd3b63c3fa':
  Tighten up compiler flags for aidl

tools/aidl/Android.mk

@@ -8,6 +8,17 @@ ifeq ($(TARGET_BUILD_APPS)$(filter true,$(TARGET_BUILD_PDK)),)
 LOCAL_PATH:= $(call my-dir)
 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 := \
     AST.cpp \
     Type.cpp \
@@ -22,7 +33,6 @@ LOCAL_SRC_FILES := \
     options.cpp \
     search_path.cpp \
 
-LOCAL_CFLAGS := -g
 LOCAL_MODULE := aidl
 LOCAL_MODULE_HOST_OS := darwin linux windows
 

tools/aidl/aidl.cpp

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