On Tue, 5 May 2015 14:19:44 +0100
Vittorio Giovara <[email protected]> wrote:

> On Sat, May 2, 2015 at 5:50 AM, Anton Khirnov <[email protected]> wrote:
> > Quoting Vittorio Giovara (2015-05-02 01:17:17)
> >> Add an helper function and a compatibility layer for exporting
> >> information through coded_frame.
> >>
> >> The compression data is exported to the decoded frame, offering a direct
> >> correspondece between the packet and the frame properties.
> >>
> >> Signed-off-by: Vittorio Giovara <[email protected]>
> >> ---
> >>  doc/APIchanges        |  5 +++++
> >>  libavcodec/avcodec.h  |  6 ++++++
> >>  libavcodec/internal.h |  5 +++++
> >>  libavcodec/utils.c    | 33 +++++++++++++++++++++++++++++++++
> >>  libavcodec/version.h  |  4 ++--
> >>  libavformat/dump.c    | 24 ++++++++++++++++++++++++
> >>  libavutil/frame.h     |  7 +++++++
> >>  libavutil/metadata.h  | 20 ++++++++++++++++++++
> >>  libavutil/version.h   |  2 +-
> >>  9 files changed, 103 insertions(+), 3 deletions(-)
> >>
> >
> > * why is there a frame side data type? This stuff only applies to
> >   compressed packets.
> 
> because you might be interested in the compression properties that
> came from the packet.
> It's kind-of already implemented the equivalent fields in avframe too.
> 
> > * the name 'AVStats' is incredibly generic and also misleading, since
> >   there is no real statistics in it, just some per-packet information.
> >   AVPacketCompressionParam might work, but is still kind of ugly and
> >   long. Better suggestions welcome.
> 
> AVCompressionProps?
> 

Nice bikeshed you have here. Would be awful if something happened to
it...

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

Reply via email to