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

Commit 28139d9a authored by Taehwan Kim's avatar Taehwan Kim Committed by Cherrypicker Worker
Browse files

codec2 components: fix subscribed indices setter 

Bug: 222205463
Test: boot
Change-Id: Iaf4c1e0f26fe6091d8d5de1dc9e75d157fd6c222
(cherry picked from commit 4a3850ec)
Merged-In: Iaf4c1e0f26fe6091d8d5de1dc9e75d157fd6c222
parent c1e26474

media/codec2/components/base/SimpleC2Interface.cpp

+9 −1
Original line number Diff line number Diff line
@@ -28,6 +28,14 @@ namespace android { 


/* SimpleInterface */



static C2R SubscribedParamIndicesSetter(

        bool mayBlock, C2InterfaceHelper::C2P<C2SubscribedParamIndicesTuning> &me) {

    (void)mayBlock;

    (void)me;



    return C2R::Ok();

}



SimpleInterface<void>::BaseParams::BaseParams(

        const std::shared_ptr<C2ReflectorHelper> &reflector,

        C2String name,
@@ -186,7 +194,7 @@ SimpleInterface<void>::BaseParams::BaseParams( 
            .withDefault(C2SubscribedParamIndicesTuning::AllocShared(0u))

            .withFields({ C2F(mSubscribedParamIndices, m.values[0]).any(),

                          C2F(mSubscribedParamIndices, m.values).any() })

            .withSetter(Setter<C2SubscribedParamIndicesTuning>::NonStrictValuesWithNoDeps)

            .withSetter(SubscribedParamIndicesSetter)

            .build());



    /* TODO

media/codec2/components/base/include/SimpleC2Interface.h

+1 −0
Original line number Diff line number Diff line
@@ -209,6 +209,7 @@ struct Setter { 
        return me.F(me.v.value).validatePossible(me.v.value);

    }



    // TODO(b/230146771): fix crash

    static C2R NonStrictValuesWithNoDeps(

            bool mayBlock, C2InterfaceHelper::C2P<type> &me) {

        (void)mayBlock;