[FFmpeg-devel] [PATCH] more ALAC cleanup

Michael Niedermayer michaelni
Wed Jul 18 17:24:09 CEST 2007


Hi

On Wed, Jul 18, 2007 at 05:19:50PM +0200, Vitor Sessak wrote:
> Hi
> 
> Michael Niedermayer wrote:
> > Hi
> > 
> > On Wed, Jul 18, 2007 at 08:35:02AM +0200, Vitor Sessak wrote:
> >> Hi
> >>
> >> This patch removes a custom inefficient implementation of av_log2 to use 
> >> FFmpeg one.  Please comment.
> > 
> > ok
> 
> Applied.
> 
> Just a question: is the following:
> 
> predictor_coef_table[chan][i] = (int16_t)get_bits(&alac->gb, 16);
> 
> safe for little-endian systems? When I searched the list, it seems that 
> since a patch applied an year ago ALAC decoding works fine in 
> little-endian systems...

i see nothing wrong with it ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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-devel/attachments/20070718/a3e39517/attachment.pgp>



More information about the ffmpeg-devel mailing list