@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/[email protected]>