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

Commit 54f877b5 authored by Eric Laurent's avatar Eric Laurent Committed by android-build-merger
Browse files

Merge "audio policy: fix virtual called from constructor" am: 1a5100d4 am: 0d852e4c 

am: 296036ed

Change-Id: I5be87d1cdcb55d281192de028823918fda34795e
parents fe4bf25c 296036ed

services/audiopolicy/manager/AudioPolicyFactory.cpp

+7 −1
Original line number Diff line number Diff line
@@ -21,7 +21,13 @@ namespace android { 
extern "C" AudioPolicyInterface* createAudioPolicyManager(

        AudioPolicyClientInterface *clientInterface)

{

    return new AudioPolicyManager(clientInterface);

    AudioPolicyManager *apm = new AudioPolicyManager(clientInterface);

    status_t status = apm->initialize();

    if (status != NO_ERROR) {

        delete apm;

        apm = nullptr;

    }

    return apm;

}



extern "C" void destroyAudioPolicyManager(AudioPolicyInterface *interface)

services/audiopolicy/managerdefault/AudioPolicyManager.cpp

+0 −1
Original line number Diff line number Diff line
@@ -4306,7 +4306,6 @@ AudioPolicyManager::AudioPolicyManager(AudioPolicyClientInterface *clientInterfa 
        : AudioPolicyManager(clientInterface, false /*forTesting*/)

{

    loadConfig();

    initialize();

}



void AudioPolicyManager::loadConfig() {

services/audiopolicy/managerdefault/AudioPolicyManager.h

+2 −1
Original line number Diff line number Diff line
@@ -307,6 +307,8 @@ public: 
            return volumeGroup != VOLUME_GROUP_NONE ? NO_ERROR : BAD_VALUE;

        }



        status_t initialize();



protected:

        // A constructor that allows more fine-grained control over initialization process,

        // used in automatic tests.
@@ -321,7 +323,6 @@ protected: 
        //   - initialize.

        AudioPolicyConfig& getConfig() { return mConfig; }

        void loadConfig();

        status_t initialize();



        // From AudioPolicyManagerObserver

        virtual const AudioPatchCollection &getAudioPatches() const