On Tue, Mar 15, 2011 at 8:37 AM, Keith Packard <kei...@keithp.com> wrote:
> On Tue, 15 Mar 2011 11:05:06 -0400, Adam Jackson <a...@redhat.com> wrote:
>
>> -                             if (asprintf(&s, "%ul %ul", val1, val.num) == 
>> -1)
>> +                             if (asprintf(&s, "%lu %u", val1, val.num) == 
>> -1)
>
> Reviewed-by: Keith Packard <kei...@keithp.com>
>
> (as an aside, I was amused to find that 'val' is a global symbol in the
> X server).

Well played. The xorg.conf parser holds some real gems.

--
Dan
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to