12 Feb
2010
12 Feb
'10
8:25 p.m.
+ dst_line[x] = 0xff000000 + | cliptobyte((c2 + r2) >> 8) << 16 /* red */ + | cliptobyte((c2 + g2) >> 8) << 8 /* green */ + | cliptobyte((c2 + b2) >> 8); /* blue */ I'm not sure if 0xff is the correct value for the X channel, but technically it shouldn't really matter since the X is ignored. Is there a specific reason why you set it to 0xff?
Otherwise the patch looks ok to me, but I think Henri might want to take a look at it as well before it is committed.