[FFmpeg-devel] [PATCH v3] avformat/mxfenc: add h264 profiles

Thomas Mundt tmundt75 at gmail.com
Sun May 6 22:31:53 EEST 2018


2018-05-06 13:32 GMT+02:00 Tomas Härdin <tjoppen at acc.umu.se>:

> fre 2018-05-04 klockan 01:52 +0200 skrev Thomas Mundt:
> > Hi,
> >
> > this is a better version of the patch.
> > 10 bit and TFF are mandatory for AVC Intra only. Other profiles
> > differ.
> >
>
> > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> > index 3bb7032..81513dc 100644
> > --- a/libavformat/mxfenc.c
> > +++ b/libavformat/mxfenc.c
> > @@ -1947,22 +1947,31 @@ static const struct {
> >      int frame_size;
> >      int profile;
> >      uint8_t interlaced;
> > +    int long_gop;
>
> A comment here explaining the difference between -1, 0 and 1 would be
> nice. The rest looks OK, but I didn't read the relevant specs to be
> 100% sure
> <http://ffmpeg.org/mailman/listinfo/ffmpeg-devel>
>

New patch attached.

Regards,
Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avformat-mxfenc-add-h264-profiles.patch
Type: application/octet-stream
Size: 8617 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180506/550fa21e/attachment.obj>


More information about the ffmpeg-devel mailing list