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

Nicolas George nicolas.george at normalesup.org
Mon Feb 27 10:35:53 CET 2012

L'octidi 8 ventôse, an CCXX, Carl Eugen Hoyos a écrit :
> 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.

If I understand you correctly, the reason for the reverts is: they had
become redundant due to a recent merge from the fork?

If so, this is a perfectly valid reason, of course.


  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20120227/1d17adca/attachment.asc>

More information about the ffmpeg-cvslog mailing list