On Mon, Oct 21, 2013 at 6:01 PM, <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > We store cursor_x/y as int16_t internally, but the user provided > coordinates are int32_t. Clamp the coordinates so that they don't > overflow the int16_t. Since the cursor is only 64x64 in size, the > clamping can't cause any visual changes. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> For shits&giggles: You've bothered with a subtest for this in your crc cursor test? Should be a one-liner and would neatly test whether QA catches it ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
