[FFmpeg-devel] [PATCH 1 of 3] movenc: enable writing of interlace information back to the 'fiel' atom. (2nd Version)

[Tomas Härdin]

On Thu, 2012-11-01 at 14:31 +0000, Tim Nicholson wrote:
> On 01/11/12 13:57, Tomas Härdin wrote:
> > On Thu, 2012-11-01 at 12:05 +0000, Tim Nicholson wrote:
> >> On 26/10/12 13:52, Tim Nicholson wrote:
> >>> On 26/10/12 12:34, Tomas Härdin wrote:
> >>>>> +    /* If field_order has set by the coder to indicate interlace coding
> >>>>> +     * update value to reflect current coded top_field_first status */
> >>>>> +    if ((track->enc->coded_frame) && (track->enc->field_order > AV_FIELD_PROGRESSIVE))
> >>>>
> >>>> Useless parentheses
> >>>
> >>> Oops, forgot to clean them out when I changed the test.
> >>
> >> No other commentsa so fixed and rebased, ping.
> > 
> > Put this somewhere in utils.c instead other muxers get good values too?
> > I'm thinking mxfenc.c.
> > 
> 
> That uses an entirely different flag for interlace within
> AVFormatContext->priv_data

Well, mxfenc parses DV and MPEG-2 essence to figure out the interlacing
stuff. I'd prefer if the MXF and MOV muxers figured out this information
the same way, whichever method is used.

/Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121102/0f0cefcd/attachment.asc>