[geany/geany-plugins] f7cf5d: Merge pull request #644 from eht16/fix_cppcheck_warnings

[Frank Lanitz]

Branch:      refs/heads/master
Author:      Frank Lanitz <frank at frank.uvena.de>
Committer:   GitHub <noreply at github.com>
Date:        Fri, 10 Nov 2017 23:16:24 UTC
Commit:      f7cf5d6dc9bff5d2d5f2f52ab4b2fdf5848ad2df
             https://github.com/geany/geany-plugins/commit/f7cf5d6dc9bff5d2d5f2f52ab4b2fdf5848ad2df

Log Message:
-----------
Merge pull request #644 from eht16/fix_cppcheck_warnings

Fix cppcheck warnings


Modified Paths:
--------------
    projectorganizer/src/prjorg-menu.c
    xmlsnippets/src/Makefile.am
    xmlsnippets/src/tests.c

Modified: projectorganizer/src/prjorg-menu.c
2 lines changed, 1 insertions(+), 1 deletions(-)
===================================================================
@@ -114,7 +114,7 @@ static void on_swap_header_source(G_GNUC_UNUSED GtkMenuItem * menuitem, G_GNUC_U
 	if (known_type)
 	{
 		gboolean swapped;
-		GSList *elem, *list = NULL;
+		GSList *elem = NULL, *list = NULL;
 		guint i = 0;
 
 		foreach_document(i)


Modified: xmlsnippets/src/Makefile.am
1 lines changed, 1 insertions(+), 0 deletions(-)
===================================================================
@@ -21,5 +21,6 @@ xmlsnippets_check_SOURCES = \
 
 xmlsnippets_check_CPPFLAGS = -DTEST
 xmlsnippets_check_LDADD = $(COMMONLIBS)
+TESTS = $(check_PROGRAMS)
 
 include $(top_srcdir)/build/cppcheck.mk


Modified: xmlsnippets/src/tests.c
2 lines changed, 1 insertions(+), 1 deletions(-)
===================================================================
@@ -80,7 +80,7 @@ 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 (g_strcmp0(c.completion, completion_needed) != 0)
 	{
 		g_warning(
 			"Test %d: FAIL\n"



--------------
This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).