Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ecaf21c3 authored by Christopher Wiley's avatar Christopher Wiley
Browse files

Revert "Tighten up compiler flags for aidl" 

This reverts commit eea2353a.

This seems to have broken the build because whatever compiler the builder is using doesn't understand the same "turn off this warning" flags as my local host's compiler.

Change-Id: Ic68b7065b161a5ae683b7082c67147835833e0fa
parent eea2353a

tools/aidl/Android.mk

+1 −11
Original line number Diff line number Diff line
@@ -8,17 +8,6 @@ ifeq ($(TARGET_BUILD_APPS)$(filter true,$(TARGET_BUILD_PDK)),) 
LOCAL_PATH:= $(call my-dir)

include $(CLEAR_VARS)



LOCAL_CFLAGS := -g -Wall -Werror

# Tragically, the code is riddled with unused parameters.

LOCAL_CFLAGS += -Wno-unused-parameter

# yacc dumps a lot of code *just in case*.

LOCAL_CFLAGS += -Wno-unused-function

LOCAL_CFLAGS += -Wno-unneeded-internal-declaration

# yacc is a tool from a more civilized age.

LOCAL_CFLAGS += -Wno-deprecated-register

# yacc also has a habit of using char* over const char*.

LOCAL_CFLAGS += -Wno-writable-strings



LOCAL_SRC_FILES := \

	aidl_language_l.l \

	aidl_language_y.y \
@@ -32,6 +21,7 @@ LOCAL_SRC_FILES := \ 
	generate_java_binder.cpp \

	generate_java_rpc.cpp



LOCAL_CFLAGS := -g

LOCAL_MODULE := aidl



include $(BUILD_HOST_EXECUTABLE)

tools/aidl/aidl.cpp

+5 −0
Original line number Diff line number Diff line
@@ -165,6 +165,11 @@ 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)