[FFmpeg-devel] [PATCH] Yet more ALAC cleanup (#define removal)

Michael Niedermayer michaelni
Thu Aug 30 18:37:00 CEST 2007


Hi

On Thu, Aug 30, 2007 at 06:05:05PM +0200, Vitor Sessak wrote:
> Hi
> 
> Michael Niedermayer wrote:
> > Hi
> > 
> >>
> >> But sign_extended32 is called with negative values as parameters (!) so 
> > 
> > ok, i see the problem, the needed operation is not really extending the
> > sign but rather truncating the msbs and then extending the remaining bit
> > so your original code is ok
> 
> Applied. Are you ok with renaming sign_extended32 for extend_sign32 ?

ok

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

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- 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-devel/attachments/20070830/37b1107d/attachment.pgp>



More information about the ffmpeg-devel mailing list