On 26/02/14 08:58 PM, Boucman wrote:
> In src/libs/masks.c:
>
>> @@ -995,7 +995,7 @@ static int _tree_button_pressed (GtkWidget *treeview, GdkEventButton *event, dt_
>>            }
>>            if (nbuse != -1)
>>            {
>> -            if (nbuse>0) strcat(str," )");
>> +            if (nbuse>0) g_strlcat(str, ") ", sizeof(str));
>>
>
> you added a whitespace here

I don't see a problem with that. To my way of thinking, it
improves readability. I'd put the g_strlcat(..) on a separate
line, but apart from that I have no issues with it.


James C. McPherson
--
Solaris kernel software engineer, system admin and troubleshooter
               https://www.jmcpdotcom.com/blog
Find me on LinkedIn @ http://www.linkedin.com/in/jamescmcpherson