[Geany-Devel] Gaeny-plugins: Some minor changes to silence warnings

Colomban Wendling lists.ban at xxxxx
Tue Feb 19 13:31:28 UTC 2013


Le 19/02/2013 13:22, Frank Lanitz a écrit :
> Am 17.02.2013 04:09, schrieb Lex Trotman:
>> Attached is a patch containing a few obvious fixes for warnings in the
>> plugins, mostly adding consts, gpointer casts and unsigned int loop
>> counters.
>>
>> The only one that is likely to have been bad (tm) is in treebrowser.c
>> it g_freed a static string.  Setting the string const found that.
>>
>> treebrowser.c still has warnings about two unused variables, since in
>> each case some work is done to set them, the treebrowser maintainer
>> should look and see if they are meant to be used somewhere, or remove
>> them and the setting code.
> 
> I had a short look and think we can apply it. At least I didn't see
> anything which is likely to fail.

I didn't test it, but agree that the diff looks fine.

Cheers,
Colomban


More information about the Devel mailing list