On Wed, 31 Dec 2014 01:35:59 +0530
Anshul <anshul.ffm...@gmail.com> wrote:

> From 4e95ec0923af73a07ae9bacb1f371f43be93ca61 Mon Sep 17 00:00:00 2001
> From: Anshul Maheshwari <anshul.ffm...@gmail.com>
> Date: Wed, 31 Dec 2014 01:23:20 +0530
> Subject: [PATCH] handle scte 35 message cue data stream
> 



>  
>  /**
> + * Decode a subtitle message.
> + * Return a negative value on error, otherwise return the number of bytes 
> used.
> + * If no data was decompressed then size parameter of output is zero
> + * 
> + * @param avctx the codec context
> + * @param[out] output The Preallocated AVBuffer in which the decoded data 
> will
> + *                    be stored, must be freed with free parameter of  
> AVBuffer
> + *                    if size is not 0.
> + * @param[in] avpkt The input AVPacket containing the input buffer.
> + */
> +int avcodec_decode_data(AVCodecContext *avctx,
> +                                              AVBuffer *output,
> +                                              const AVPacket *avpkt);
> +/**


It's still not clear what the function actually does. What's the output
format? Why is avcodec_decode_subtitle2() not used for this? (I'm being
naive and dumb here; while I don't know what this patch is actually
about, users trying to understand the lavc APIs in general will have
the same questions.)

Also, it mixes the words "decode" and "decompress".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to