[FFmpeg-devel] [PATCH]Reorder channels for some mov channel layouts

Carl Eugen Hoyos cehoyos at ag.or.at
Wed Jun 15 14:41:57 CEST 2011

Nicolas George <nicolas.george <at> normalesup.org> writes:

> Le quintidi 25 prairial, an CCXIX, Carl Eugen Hoyos a écrit :
> > +    /**
> > +     * mov channel layout tag as described in the CAF specification.
> > +     * - decoding: Set by user.
> > +     * - encoding: unused
> > +     */
> > +    int32_t mov_channel_layout;
> Do we really want MOV-specific fields in this structure? Couldn't it be
> translated in a more universal channel_layout value in isom.c?

I don't know how this should be possible but please make a suggestion.
Don't forget that re-muxing should still be possible (the "unused" above is just
to make this patch as small as possible, the follow-up patch changes it to "Set
by user, overwritten by (AAC) encoder." and allows to write AAC channel layout
in mov and to remux the sample from issue 98 into aiff).

What about my reordering code?

Thank you, Carl Eugen

More information about the ffmpeg-devel mailing list