On Mon, 23 Nov 2009 03:03:01 +0100, Alan Coopersmith  
<[email protected]> wrote:

> Kim Woelders wrote:
>> -    char *pattern = NULL, *ptr = NULL;
>> +    char *pattern, *ptr;
>
> Why remove the initialization to NULL?
>
They are useless assignments. There are other useless initializations  
(pixel_size, field_num, len) but they are not related to the fix, which is  
about properly freeing the pattern buffer, so it didn't feel right to  
change those.

Anyway, removing the initializations or not makes no difference in the end.

/Kim

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to