[FFmpeg-devel] [PATCH] lavc: add pkt_size field to AVFrame

[Michael Niedermayer]

On Fri, Dec 14, 2012 at 05:52:16PM +0100, Stefano Sabatini wrote:
> TODO: add APIchanges entry, bump minor
> ---
>  libavcodec/avcodec.h |   12 ++++++++++++
>  libavcodec/utils.c   |    4 ++++
>  2 files changed, 16 insertions(+)
> 
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index e432040..2c8288b 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -1467,6 +1467,16 @@ typedef struct AVFrame {
>       * - decoding: Read by user.
>       */
>      int64_t channels;
> +
> +    /**
> +     * size of the corresponding packet, containing the compressed
> +     * size of the frame, only relevant for video. It must be accessed
> +     * using av_frame_get_pkt_size() and av_frame_set_pkt_size().

> +     * It is set to -1 if unknown.

i suggest negative here (under the principle of leaving the API
maximally flexible for future extension)

and a entry should be added to frame_options

otherwise this should be ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121215/395c6416/attachment.asc>