On Tue, Jun 14, 2011 at 02:11:42PM +1000, Peter Hutterer wrote:
> On Thu, Jun 09, 2011 at 08:57:22PM +0100, Daniel Stone wrote:
> > From: Derek Foreman <[email protected]>
> > 
> > UINT32, not float.
> 
> not that a negative value would make sense here but the driver (and your
> code) uses int internally so a commit message fixup would be in order.
> in this case we should also check for negative values. or you switch this
> patch to use CARD32. 

Ah yep, I'll make it CARD32.

Cheers,
Daniel
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to