Le 25/09/2011 18:59, fr...@users.sourceforge.net a écrit :
> Revision: 2214
>           
> http://geany-plugins.svn.sourceforge.net/geany-plugins/?rev=2214&view=rev
> Author:   frlan
> Date:     2011-09-25 16:59:59 +0000 (Sun, 25 Sep 2011)
> Log Message:
> -----------
> GeanySendMail: Adding a workaround for a weird issue in connection with 
> utils_string_replace_all().
> [...]
> @@ -149,8 +148,11 @@
>                       else
>                       {
>                               /* Removes %r if option was not activ but was 
> included into command */
> -                             utils_string_replace_all(cmd_str, "%r", NULL);
> +                             gchar *tmp_fix;
> +                             tmp_fix = g_strdup("");
> +                             utils_string_replace_all(cmd_str, "%r", 
> tmp_fix);
>                               g_free(address);
> +                             g_free(tmp_fix);

No need to duplicate tmp_fix, passing "" directly is OK, a constant
string is expected:

utils_string_replace_all(cmd_str, "%r", "");

Cheers,
Colomban
_______________________________________________
Geany-devel mailing list
Geany-devel@uvena.de
https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel

Reply via email to