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

Commit 59bb1946 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "CodecBase: fix parameter query methods" into sc-dev am: 705168c2 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/15367052

Change-Id: I7678b692aeae9b0620d15a48cf0337ca5acea29a
parents 3db50af3 705168c2

media/libstagefright/CodecBase.cpp

+27 −0
Original line number Diff line number Diff line
@@ -40,4 +40,31 @@ void BufferChannelBase::IMemoryToSharedBuffer( 
    buf->size = size;

}



status_t CodecBase::querySupportedParameters(std::vector<std::string> *names) {

    if (names == nullptr) {

        return BAD_VALUE;

    }

    names->clear();

    return OK;

}



status_t CodecBase::describeParameter(const std::string &, CodecParameterDescriptor *) {

    return ERROR_UNSUPPORTED;

}



status_t CodecBase::subscribeToParameters(const std::vector<std::string> &names) {

    if (names.empty()) {

        return OK;

    }

    return ERROR_UNSUPPORTED;

}



status_t CodecBase::unsubscribeFromParameters(const std::vector<std::string> &names) {

    if (names.empty()) {

        return OK;

    }

    return ERROR_UNSUPPORTED;

}





} // namespace android

media/libstagefright/include/media/stagefright/CodecBase.h

+5 −15
Original line number Diff line number Diff line
@@ -252,9 +252,7 @@ struct CodecBase : public AHandler, /* static */ ColorUtils { 
     *         INVALID_OPERATION if already released;

     *         ERROR_UNSUPPORTED if not supported.

     */

    virtual status_t querySupportedParameters([[maybe_unused]] std::vector<std::string> *names) {

        return ERROR_UNSUPPORTED;

    }

    virtual status_t querySupportedParameters(std::vector<std::string> *names);

    /**

     * Fill |desc| with description of the parameter with |name|.

     *
@@ -267,10 +265,8 @@ struct CodecBase : public AHandler, /* static */ ColorUtils { 
     *         ERROR_UNSUPPORTED if not supported.

     */

    virtual status_t describeParameter(

            [[maybe_unused]] const std::string &name,

            [[maybe_unused]] CodecParameterDescriptor *desc) {

        return ERROR_UNSUPPORTED;

    }

            const std::string &name,

            CodecParameterDescriptor *desc);

    /**

     * Subscribe to parameters in |names| and get output format change event

     * when they change.
@@ -281,10 +277,7 @@ struct CodecBase : public AHandler, /* static */ ColorUtils { 
     *         INVALID_OPERATION if already released;

     *         ERROR_UNSUPPORTED if not supported.

     */

    virtual status_t subscribeToParameters(

            [[maybe_unused]] const std::vector<std::string> &names) {

        return ERROR_UNSUPPORTED;

    }

    virtual status_t subscribeToParameters(const std::vector<std::string> &names);

    /**

     * Unsubscribe from parameters in |names| and no longer get

     * output format change event when they change.
@@ -295,10 +288,7 @@ struct CodecBase : public AHandler, /* static */ ColorUtils { 
     *         INVALID_OPERATION if already released;

     *         ERROR_UNSUPPORTED if not supported.

     */

    virtual status_t unsubscribeFromParameters(

            [[maybe_unused]] const std::vector<std::string> &names) {

        return ERROR_UNSUPPORTED;

    }

    virtual status_t unsubscribeFromParameters(const std::vector<std::string> &names);



    typedef CodecBase *(*CreateCodecFunc)(void);

    typedef PersistentSurface *(*CreateInputSurfaceFunc)(void);