[FFmpeg-cvslog] r12871 - trunk/libavcodec/alac.c

Michael Niedermayer michaelni
Thu Apr 17 20:43:36 CEST 2008


On Thu, Apr 17, 2008 at 08:20:25PM +0200, Vitor Sessak wrote:
> Vitor Sessak wrote:
>> michael wrote:
>>> Author: michael
>>> Date: Thu Apr 17 04:40:50 2008
>>> New Revision: 12871
>>>
>>> Log:
>>> more senseless code
>>> One wonders how such stuff gets past the reviews ...

>> I suppose that you remember that one year ago, the main decoding loop was 
>> triplicated (!): once for mono and once for each stereo channel.

no i dont remember, which commit was that exactly?


>> Well, since you are at it, the attached patch removes a unneeded wrapper 
>> around av_log2().
>
> Oops, I missed an obvious simplification. This one is better.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080417/1f06eaa3/attachment.pgp>



More information about the ffmpeg-cvslog mailing list