SF.net SVN: geany:[3891] trunk

[ntrel at users.sourceforge.net]

Revision: 3891
          http://geany.svn.sourceforge.net/geany/?rev=3891&view=rev
Author:   ntrel
Date:     2009-06-23 12:51:16 +0000 (Tue, 23 Jun 2009)

Log Message:
-----------
Add debug message if plugin has not set a name for its keybinding
group.

Modified Paths:
--------------
    trunk/ChangeLog
    trunk/src/plugins.c

Modified: trunk/ChangeLog
===================================================================
--- trunk/ChangeLog	2009-06-22 15:26:44 UTC (rev 3890)
+++ trunk/ChangeLog	2009-06-23 12:51:16 UTC (rev 3891)
@@ -1,3 +1,10 @@
+2009-06-23  Nick Treleaven  <nick(dot)treleaven(at)btinternet(dot)com>
+
+ * src/plugins.c:
+   Add debug message if plugin has not set a name for its keybinding
+   group.
+
+
 2009-06-22  Nick Treleaven  <nick(dot)treleaven(at)btinternet(dot)com>
 
  * src/highlighting.c, doc/geany.txt, doc/geany.html,

Modified: trunk/src/plugins.c
===================================================================
--- trunk/src/plugins.c	2009-06-22 15:26:44 UTC (rev 3890)
+++ trunk/src/plugins.c	2009-06-23 12:51:16 UTC (rev 3891)
@@ -484,7 +484,13 @@
 {
 	guint i;
 
-	g_return_if_fail(NZV(plugin->key_group->name));
+	if (!NZV(plugin->key_group->name))
+	{
+		geany_debug("Plugin \"%s\" has not set a name for its keybinding group"
+			" - ignoring all keybindings!",
+			plugin->info.name);
+		return;
+	}
 	g_return_if_fail(! g_str_equal(plugin->key_group->name, keybindings_keyfile_group_name));
 
 	for (i = 0; i < plugin->key_group->count; i++)


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.