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

Commit bfb9b0d1 authored by Shunkai Yao's avatar Shunkai Yao
Browse files

Make EffectsFactoryHalHidl members const 

Bug: 261129656
Test: m
Change-Id: I57809e61463c6420855f9f37a436fba369e5d14b
parent e3e403d4

media/libaudiohal/impl/EffectsFactoryHalHidl.cpp

+12 −10
Original line number Diff line number Diff line
@@ -80,18 +80,20 @@ EffectDescriptorCache::QueryResult EffectDescriptorCache::queryAllDescriptors( 


EffectsFactoryHalHidl::EffectsFactoryHalHidl(sp<IEffectsFactory> effectsFactory)

    : EffectConversionHelperHidl("EffectsFactory"),

      mEffectsFactory(std::move(effectsFactory)),

      mCache(new EffectDescriptorCache),

      mEffectProcessings([&]() -> effectsConfig::EffectProcessings {

      mEffectProcessings(

              [&]() -> effectsConfig::EffectProcessings {

                  effectsConfig::EffectProcessings processings;

                  const auto& parseResult = effectsConfig::parse();

                  if (!parseResult.parsedConfig) {

                      return INVALID_EFFECT_PROCESSING;

                  }

                  return {parseResult.nbSkippedElement, parseResult.parsedConfig->preprocess,

                  parseResult.parsedConfig->postprocess, parseResult.parsedConfig->deviceprocess};

                          parseResult.parsedConfig->postprocess,

                          parseResult.parsedConfig->deviceprocess};

              }()) {

    ALOG_ASSERT(effectsFactory != nullptr, "Provided IEffectsFactory service is NULL");

    mEffectsFactory = std::move(effectsFactory);

}



status_t EffectsFactoryHalHidl::queryNumberEffects(uint32_t *pNumEffects) {

media/libaudiohal/impl/EffectsFactoryHalHidl.h

+2 −2
Original line number Diff line number Diff line
@@ -66,8 +66,8 @@ class EffectsFactoryHalHidl final : public EffectsFactoryHalInterface, 
    const effectsConfig::EffectProcessings& getProcessings() const override;



  private:

    sp<IEffectsFactory> mEffectsFactory;

    std::unique_ptr<EffectDescriptorCache> mCache;

    const sp<IEffectsFactory> mEffectsFactory;

    const std::unique_ptr<EffectDescriptorCache> mCache;

    // Configuration file parser result together with all processings from effect factory

    const effectsConfig::EffectProcessings mEffectProcessings;

};