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