[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