[Ffmpeg-devel] H.264 encoder

[Reimar Döffinger]

Hello,
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 :)

Sure,
val -= (val >> 14) | 1;
would do it, too, but that's not a bug? Though the compiler might
complain a bit (but hopefully still produce correct code).

Greetings,
Reimar D?ffinger