[Github-comments] [geany/geany-plugins] Fix cppcheck warnings (#644)

Enrico Tröger notifications at xxxxx
Tue Nov 7 06:11:42 UTC 2017


eht16 commented on this pull request.



> @@ -80,7 +80,9 @@ static gboolean test(gint ordinal, const gchar *input, const gchar *completion_n
 	else if (!completion_needed && !success)
 		return TRUE;
 
-	if (strcmp(c.completion, completion_needed) != 0)
+	if ((!completion_needed && c.completion) ||
+		(completion_needed && !c.completion) ||
+		strcmp(c.completion, completion_needed) != 0)

Of course, did it and re-pushed.

-- 
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-plugins/pull/644#discussion_r149279172
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20171107/c9c53e1b/attachment.html>


More information about the Github-comments mailing list