[Ffmpeg-devel] H.264 encoder
Thu Oct 5 16:53:16 CEST 2006
On Thu, 2006-10-05 at 16:43 +0200, Panagiotis Issaris wrote:
> On Thu, Oct 05, 2006 at 05:34:26PM +0300, Uoti Urpala wrote:
> > On Thu, 2006-10-05 at 15:35 +0200, Panagiotis Issaris wrote:
> > > + int16_t val = levels[i];
> > > + val -= (val>>31)|1;
> > Seems you didn't spot the bug in Michael's recommendation :)
> And I still don't :) I just rechecked and it seems to be working...
Actually it should work that way too (so it's not really a bug), but I
think it would be better to use (val>>15) with a 16-bit variable.
More information about the ffmpeg-devel