[FFmpeg-cvslog] r8960 - trunk/libavcodec/pcm.c
Michael Niedermayer
michaelni
Thu May 10 01:58:51 CEST 2007
Hi
On Thu, May 10, 2007 at 01:08:01AM +0200, ramiro wrote:
> Author: ramiro
> Date: Thu May 10 01:08:01 2007
> New Revision: 8960
>
> Log:
> Factorize usum
are you pcm.c maintainer? no?
did you send a patch? no?
is this change doing anything good besides bloating the code?
did you benchmark the code?
if the change would make sense i wouldnt mind if you commited without
sending a patch but it doesnt, at least its not obvious what good it does
these functions are static inline and the compiler will if it inlines
them optimize the check away, and the functions are small so i guess
gcc will inline them
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070510/8382e793/attachment.pgp>
More information about the ffmpeg-cvslog
mailing list