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

Commit 3427ab41 authored by Caroline Tice's avatar Caroline Tice
Browse files

Fix 2 compiler warnings. 

This CL fixes the following 2 compiler warnings:

EffectBundle.cpp:343:18 warning: Dereference of null pointer
  (loaded from variable 'pHandle')
EffectBundle.cpp:2662:13 warning: Value stored to 'status' is never read

BUG: None
Test: None
Change-Id: Ifa68a79c6fe82ffcbd22890e1187c92a72535b7e
parent 93f52337

media/libeffects/lvm/wrapper/Bundle/EffectBundle.cpp

+5 −3
Original line number Diff line number Diff line
@@ -340,8 +340,10 @@ exit: 
            }

            delete pContext;

        }

        if (pHandle != NULL)

          *pHandle = (effect_handle_t)NULL;

    } else {

      if (pHandle != NULL)

        *pHandle = (effect_handle_t)pContext;

    }

    ALOGV("\tEffectCreate end..\n\n");
@@ -2661,8 +2663,8 @@ int Volume_setParameter (EffectContext *pContext, void *pParam, void *pValue){ 


        case VOLUME_PARAM_ENABLESTEREOPOSITION:

            positionEnabled = *(uint32_t *)pValue;

            status = VolumeEnableStereoPosition(pContext, positionEnabled);

            status = VolumeSetStereoPosition(pContext, pContext->pBundledContext->positionSaved);

            (void) VolumeEnableStereoPosition(pContext, positionEnabled);

            (void) VolumeSetStereoPosition(pContext, pContext->pBundledContext->positionSaved);

            //ALOGV("\tVolume_setParameter() VOLUME_PARAM_ENABLESTEREOPOSITION called");

            break;