On Mon, Feb 27, 2012 at 11:29 PM, Diego Biurrun <[email protected]> wrote:
> On Mon, Feb 27, 2012 at 10:46:40PM +0200, Felipe Contreras wrote:
>> Notice the library is called libtidsp, and the name libtidspbridge is
>> abusrdly long, inconvenient, and inconsistent with the actual library
>> name, but that's what was requested on the mailing list.
>>
>> Signed-off-by: Felipe Contreras <[email protected]>
>> ---
>>  configure                      |    5 ++
>>  libavcodec/Makefile            |    1 +
>>  libavcodec/allcodecs.c         |    1 +
>>  libavcodec/tidspbridge_mpeg4.c |  173 
>> ++++++++++++++++++++++++++++++++++++++++
>>  4 files changed, 180 insertions(+)
>>  create mode 100644 libavcodec/tidspbridge_mpeg4.c
>
> Please add a Changelog entry.

Ok.

>> --- /dev/null
>> +++ b/libavcodec/tidspbridge_mpeg4.c
>> @@ -0,0 +1,173 @@
>> +static av_cold int init(AVCodecContext *avctx)
>> +{
>> +    struct td_av_context *ctx;
>> +    struct td_context *td_ctx;
>> +
>> +    avctx->internal->hwaccel_private = ctx = av_mallocz(sizeof(*ctx));
>> +    if (!ctx)
>> +        return -1;
>
> return AVERROR(ENOMEM);

OK.

>> +static int start_frame(AVCodecContext *avctx, const uint8_t *buffer,
>> +        uint32_t size)
>
>  static int start_frame(AVCodecContext *avctx, const uint8_t *buffer,
>                         uint32_t size)

I don't see any of that in the documentation.

Can you update the section 'Editor configuration' with the vim
cinoptions you are talking about, or at least some examples of this?

Cheers.

-- 
Felipe Contreras
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to