[Github-comments] [geany/geany] Fix new GCC warnings (#1725)
Thomas Martitz
notifications at xxxxx
Thu Dec 21 08:29:00 UTC 2017
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20171221/1c0c5ac1/attachment.html>
More information about the Github-comments
mailing list