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

Commit ef225074 authored by Glenn Kasten's avatar Glenn Kasten
Browse files

Enable strict warnings and fix unused warnings

Test: builds without warnings
Change-Id: I0701a44cbd9a2adc405a1e253b57df7ee03a90b7
parent 4e1fe188
Loading
Loading
Loading
Loading
+1 −0
Original line number Diff line number Diff line
@@ -20,5 +20,6 @@ LOCAL_C_INCLUDES := \
	$(call include-path-for, audio-utils)

LOCAL_CFLAGS += -fvisibility=hidden
LOCAL_CFLAGS += -Wall -Werror

include $(BUILD_SHARED_LIBRARY)
+0 −1
Original line number Diff line number Diff line
@@ -389,7 +389,6 @@ static int Downmix_Command(effect_handle_t self, uint32_t cmdCode, uint32_t cmdS

    downmix_module_t *pDwmModule = (downmix_module_t *) self;
    downmix_object_t *pDownmixer;
    int retsize;

    if (pDwmModule == NULL || pDwmModule->context.state == DOWNMIX_STATE_UNINITIALIZED) {
        return -EINVAL;
+1 −0
Original line number Diff line number Diff line
@@ -8,6 +8,7 @@ LOCAL_SRC_FILES:= \
	dsp/core/dynamic_range_compression.cpp

LOCAL_CFLAGS+= -O2 -fvisibility=hidden
LOCAL_CFLAGS += -Wall -Werror

LOCAL_SHARED_LIBRARIES := \
	libcutils \
+0 −2
Original line number Diff line number Diff line
@@ -194,7 +194,6 @@ int LELib_Create(const effect_uuid_t *uuid,
                         effect_handle_t *pHandle) {
    ALOGV("LELib_Create()");
    int ret;
    int i;

    if (pHandle == NULL || uuid == NULL) {
        return -EINVAL;
@@ -311,7 +310,6 @@ int LE_command(effect_handle_t self, uint32_t cmdCode, uint32_t cmdSize,
        void *pCmdData, uint32_t *replySize, void *pReplyData) {

    LoudnessEnhancerContext * pContext = (LoudnessEnhancerContext *)self;
    int retsize;

    if (pContext == NULL || pContext->mState == LOUDNESS_ENHANCER_STATE_UNINITIALIZED) {
        return -EINVAL;
+2 −0
Original line number Diff line number Diff line
@@ -120,6 +120,7 @@ LOCAL_C_INCLUDES += \
    $(LOCAL_PATH)/StereoWidening/lib

LOCAL_CFLAGS += -fvisibility=hidden
LOCAL_CFLAGS += -Wall -Werror

include $(BUILD_STATIC_LIBRARY)

@@ -177,4 +178,5 @@ LOCAL_C_INCLUDES += \
    $(LOCAL_PATH)/Common/src

LOCAL_CFLAGS += -fvisibility=hidden
LOCAL_CFLAGS += -Wall -Werror
include $(BUILD_STATIC_LIBRARY)
Loading