[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