[FFmpeg-devel] [PATCH] Fix non-rounding up to next 16-bit aligned bug in IFF decoder

Benoit Fouet benoit.fouet
Thu Apr 29 15:02:49 CEST 2010


On Wed, 28 Apr 2010 23:24:55 +0200 Sebastian Vater wrote:
> Reimar D?ffinger a ?crit :
> > On Wed, Apr 28, 2010 at 03:51:26PM +0200, Sebastian Vater wrote:
> >   
> >>> This is not how people commonly round up to 2. Most people do
> >>> (x+1)&~1, so you could do:
> >>>       
> >
> > Note I think this does not apply here, but you definitely should _not_
> > use (x+1)&~1 in FFmpeg code, use FFALIGN instead.
> >   
> 
> Fixed. See attached patch!
> 

as Reimar said, it probably does not apply here.
The usage for FFALIGN is, e.g.:
foo = FFALIGN(foo, 16);

to align foo on a 16 boundary.

Ben



More information about the ffmpeg-devel mailing list