kugel- approved this pull request. Minor issue, overall looks good
> @@ -2256,7 +2256,7 @@ static void build_load_menu_grp(GKeyFile *config, > GeanyBuildCommand **dst, gint gsize prefixlen; /* NOTE prefixlen used in macros above */ GeanyBuildCommand *dstcmd; gchar *key; - static gchar cmdbuf[3] = " "; + static gchar cmdbuf[4] = " "; I would simply remove the lengths specification. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/1725#pullrequestreview-84993330