Michel Dänzer wrote:
> On Wed, 2010-01-06 at 18:35 -0700, Brian Paul wrote: 
>> Roel Kluin wrote:
>>> backref 3: src_x
>>> ---[ vi src/gallium/auxiliary/util/u_rect.c +30 ]---
>>>
>>> void
>>> util_copy_rect(ubyte * dst,
>>>                enum pipe_format format,
>>>                unsigned dst_stride,
>>>                unsigned dst_x,
>>>                unsigned dst_y,
>>>                unsigned width,
>>>                unsigned height,
>>>                const ubyte * src,
>>>                int src_stride,
>>>                unsigned src_x, 
>>>                int src_y)
>> It's kind of weird that src_x is unsigned but src_y is signed.
>> Looks like Michel made the last change there (commit 
>> e922adbe1d6c1764968377658ea92ae6de0585db).  Michel?
> 
> I don't remember the details unfortunately. Feel free to change it if it
> doesn't make sense.

I'll make it unsigned for consistancy.

I'll also remove the useless assertions and update a comment...

-Brian


------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to