changeset 20003:26f562916f98

Comment tweaks to be closer to the version handed out via the wiki as well as removing an unused variable I neglected to remove previously (eliminates an unused variable warning, obviously).
author John Bailey <rekkanoryo@rekkanoryo.org>
date Fri, 14 Sep 2007 02:41:20 +0000
parents 6583988a57ae
children 6ec43616ba5c
files libpurple/plugins/notify_example.c
diffstat 1 files changed, 5 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/plugins/notify_example.c	Fri Sep 14 00:37:11 2007 +0000
+++ b/libpurple/plugins/notify_example.c	Fri Sep 14 02:41:20 2007 +0000
@@ -42,6 +42,8 @@
 
 static PurplePlugin *notify_example = NULL;
 
+/* The next four functions and the calls within them should cause dialog boxes to appear
+ * when you select the plugin action from the Tools->Notify Example menu */
 static void
 notify_error_cb(PurplePluginAction *action)
 {
@@ -74,6 +76,7 @@
 static void
 notify_uri_cb(PurplePluginAction *action)
 {
+	/* This one should open your web browser of choice. */
 	purple_notify_uri(notify_example, "http://www.pidgin.im/");
 }
 
@@ -81,8 +84,8 @@
 plugin_actions(PurplePlugin *plugin, gpointer context)
 {
 	GList *actions = NULL;
-	PurplePluginAction *action = NULL;
 
+	/* Here we take advantage of return values to avoid the need for a temp variable */
 	actions = g_list_prepend(actions,
 		purple_plugin_action_new("Show Error Notification", notify_error_cb));
 
@@ -104,6 +107,7 @@
 static gboolean
 plugin_load(PurplePlugin *plugin)
 {
+	/* we need a handle for all the notify calls */
 	notify_example = plugin;
 
 	return TRUE;