[PATCH] Bugfix for "Pre-validate ChangeGC XIDs": off-by-one in loop index.

Jamey Sharp jamey at minilop.net
Fri May 14 11:30:52 PDT 2010


On Fri, May 14, 2010 at 11:27 AM, Jeremy Huddleston <jeremyhu at apple.com> wrote:
> On May 14, 2010, at 11:19, Jamey Sharp wrote:
>> I'd expect to see this result with yesterday's master. Are you sure
>> you're testing with my patch? :-)
>
> Yep:
>
> 443         for (i = Ones(mask); --i; )

That's... not *my* patch. That's a patch that resembles my patch quite
strongly but does not fix anything. My patch says i--. I've
triple-checked now.

Jamey


More information about the xorg-devel mailing list