From: Mike M. <mj....@sa...> - 2011-01-21 04:22:07
|
On 01/21/2011 10:04 AM, Lucas De Marchi wrote: > Patch 001_evas.patch.txt does not look right. Besides the trailing > whitespaces, it doesn't make sense. Maybe you forgot to save your > changes before sending? - free(g); - free(g->keyname); I think the essence of the patch is that the above two lines are in the wrong order. It would be clearer if whitespace changes were not mixed with fixes. thanks, Mike |