[FFmpeg-devel] [PATCH 2/2] avformat/movenc: Support merging h264 fields into field pairs
michaelni at gmx.at
Mon Mar 10 12:15:37 CET 2014
On Mon, Mar 10, 2014 at 01:18:16PM +0900, Yusuke Nakamura wrote:
> 2014-03-10 12:48 GMT+09:00 Timothy Gu <timothygu99 at gmail.com>:
> > On Mar 9, 2014 6:59 PM, "Michael Niedermayer" <michaelni at gmx.at> wrote:
> > >
> > > Fixes Ticket3393
> > >
> > > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > > ---
> > > libavformat/movenc.c | 12 ++++++++++++
> > > libavformat/movenc.h | 3 +++
> > > 2 files changed, 15 insertions(+)
> > Missing documentation:
> > Set to 1 merge field pairs into one packet. This option is only valid for
> > H.264 codec. If the codec is not H.264, then this option would be silently
> > ignored.
> > Also I wonder if it's a good idea to print a warning if this option is set
> > but codec isn't H.264.
> > Does this work with stream copying? If so, then more documentation is
> > needed.
> > How about HEVC?
> > [...]
> > Timothy
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> This patch allows to generate files which violate 14496-15.
> According to 14496-15, a sample is an access unit.
> An access unit can contain only one primary coded picture.
> What spec allows this?
The quicktime spec probably does allow it. At least i dont see
anything in it that would disallow it.
And AFAIK some encoders like mainconcept use it, i dont have that
encoder so my knowledge here is based on people who do ...
Do you object to this patch or have suggestions on how it can be
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel