[FFmpeg-devel] [PATCH 09/13] avformat/mxfenc: Add Padding Bits

[Michael Niedermayer]

On Tue, May 08, 2018 at 12:50:16PM +0200, Tomas Härdin wrote:
> mån 2018-05-07 klockan 12:38 +0200 skrev Michael Niedermayer:
> > > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> > @@ -1228,6 +1229,10 @@ static void mxf_write_cdci_common(AVFormatContext *s, AVStream *st, const UID ke
> >      mxf_write_local_tag(pb, 1, 0x3303);
> >      avio_w8(pb, sc->color_siting);
> >  
> > +    // Padding Bits
> > +    mxf_write_local_tag(pb, 2, 0x3307);
> > +    avio_wb16(pb, 0);
> 
> I'm pretty sure there's ways of muxing v210 in mxf, so it might be
> better to say nothing here unless we're entirely sure. Probably OK
> since mxfenc can't mux raw video tho

IIUC we need this for SMPTE 386M (D-10). It lists "Padding Bits"

this needs to be updated in case rawvideo support is added with different
padding bits.

will apply

thanks


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180508/98ce4795/attachment.sig>