@eht16 commented on this pull request.


> +     }
+
+       if (g_utf8_strchr(needles, -1, 'l') != NULL && strstr(haystack->str, 
"%l")) {
+               g_string_append(errormsg, " %%l");
+       }
+
+       if (g_utf8_strchr(needles, -1, 'p') != NULL && strstr(haystack->str, 
"%p")) {
+               g_string_append(errormsg, " %%p");
+       }
+
+       if (errormsg->len > 0) {
+               g_string_prepend(errormsg, "failed to substitute");
+               ui_set_statusbar(FALSE, "%s", errormsg->str);
+       }
+
+       g_stringfree(errormsg, TRUE);

```suggestion
        g_string_free(errormsg, TRUE);
```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4250#pullrequestreview-2762742915
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/4250/review/2762742...@github.com>

Reply via email to