[FFmpeg-cvslog] Revert "Fix channel layout for some stereo flac files."

Carl Eugen Hoyos cehoyos at ag.or.at
Sun Feb 26 21:52:36 CET 2012


Michael Niedermayer <michaelni <at> gmx.at> writes:

> > After checking the spec more carefully, it seems my memory tricked me: 
> > FLAC and Vorbis really have the same channels up to 6 channels, only 
> > not in the same order, but the order does not appear in channel_layout.
> > 
> > I would really like to know why this had to be reverted.
> 
> i think carl told me to revert it but maybe i misunderstood him, it
> was more git commands than words

I committed 2ef249 to fix flac channel layout. I don't remember if there 
was a user report, it is more likely that I tested different codecs / 
container channel layouts at the time. A user reported a flac sample 
that worked fine with FFmpeg (since several months) together with my 
patch five weeks ago to the fork's bugzilla (209).
Justin (or the foundation?) decided to fix the bug differently, since 
Michael (very understandably) prefers not to have different code to 
maintain (and my "fix" may not have been complete, see the subsequent 
stereo commit), he merged 62d5f9 and my commit had to be reverted.
(Sorry, I thought this was clear for everybody.)

My only concern was that I thought my original commit was at least 
not completely wrong, that is why I asked Nicolas for clarification.

Carl Eugen



More information about the ffmpeg-cvslog mailing list