SF.net SVN: geany:[4393] trunk

ntrel at users.sourceforge.net ntrel at xxxxx
Fri Oct 30 14:09:47 UTC 2009


Revision: 4393
          http://geany.svn.sourceforge.net/geany/?rev=4393&view=rev
Author:   ntrel
Date:     2009-10-30 14:09:46 +0000 (Fri, 30 Oct 2009)

Log Message:
-----------
Use typedef instead of pointer for widget_id function arguments.

Modified Paths:
--------------
    trunk/ChangeLog
    trunk/src/stash.c
    trunk/src/stash.h

Modified: trunk/ChangeLog
===================================================================
--- trunk/ChangeLog	2009-10-30 06:54:53 UTC (rev 4392)
+++ trunk/ChangeLog	2009-10-30 14:09:46 UTC (rev 4393)
@@ -1,3 +1,9 @@
+2009-10-30  Nick Treleaven  <nick(dot)treleaven(at)btinternet(dot)com>
+
+ * src/stash.c, src/stash.h:
+   Use typedef instead of pointer for widget_id function arguments.
+
+
 2009-10-29  Nick Treleaven  <nick(dot)treleaven(at)btinternet(dot)com>
 
  * src/pluginutils.c:

Modified: trunk/src/stash.c
===================================================================
--- trunk/src/stash.c	2009-10-30 06:54:53 UTC (rev 4392)
+++ trunk/src/stash.c	2009-10-30 14:09:46 UTC (rev 4393)
@@ -72,7 +72,7 @@
 	const gchar *key_name;
 	gpointer default_value;
 	GType widget_type;			/* e.g. GTK_TYPE_TOGGLE_BUTTON */
-	gpointer widget_id;			/* can be GtkWidget or gchararray */
+	GeanyWidgetID widget_id;	/* (GtkWidget*) or (gchar*) */
 	gpointer fields;			/* extra fields */
 };
 
@@ -86,7 +86,7 @@
 
 typedef struct EnumWidget
 {
-	gpointer widget_id;
+	GeanyWidgetID widget_id;
 	gint enum_id;
 }
 EnumWidget;
@@ -441,7 +441,7 @@
 
 
 static GtkWidget *
-get_widget(GtkWidget *owner, gpointer widget_id)
+get_widget(GtkWidget *owner, GeanyWidgetID widget_id)
 {
 	GtkWidget *widget = widget_id;
 
@@ -588,7 +588,7 @@
 static GeanyPrefEntry *
 add_widget_pref(GeanyPrefGroup *group, GType setting_type, gpointer setting,
 		const gchar *key_name, gpointer default_value,
-		GType widget_type, gpointer widget_id)
+		GType widget_type, GeanyWidgetID widget_id)
 {
 	GeanyPrefEntry *entry =
 		add_pref(group, setting_type, setting, key_name, default_value);
@@ -602,7 +602,7 @@
 /* Used for GtkCheckButton or GtkToggleButton widgets.
  * @see stash_group_add_radio_buttons(). */
 void stash_group_add_toggle_button(GeanyPrefGroup *group, gboolean *setting,
-		const gchar *key_name, gboolean default_value, gpointer widget_id)
+		const gchar *key_name, gboolean default_value, GeanyWidgetID widget_id)
 {
 	add_widget_pref(group, G_TYPE_BOOLEAN, setting, key_name, GINT_TO_POINTER(default_value),
 		GTK_TYPE_TOGGLE_BUTTON, widget_id);
@@ -618,7 +618,7 @@
  * @endcode */
 void stash_group_add_radio_buttons(GeanyPrefGroup *group, gint *setting,
 		const gchar *key_name, gint default_value,
-		gpointer widget_id, gint enum_id, ...)
+		GeanyWidgetID widget_id, gint enum_id, ...)
 {
 	GeanyPrefEntry *entry =
 		add_widget_pref(group, G_TYPE_INT, setting, key_name, GINT_TO_POINTER(default_value),
@@ -663,7 +663,7 @@
 
 
 void stash_group_add_spin_button_integer(GeanyPrefGroup *group, gint *setting,
-		const gchar *key_name, gint default_value, gpointer widget_id)
+		const gchar *key_name, gint default_value, GeanyWidgetID widget_id)
 {
 	add_widget_pref(group, G_TYPE_INT, setting, key_name, GINT_TO_POINTER(default_value),
 		GTK_TYPE_SPIN_BUTTON, widget_id);
@@ -672,7 +672,7 @@
 
 /* @see stash_group_add_combo_box_entry(). */
 void stash_group_add_combo_box(GeanyPrefGroup *group, gint *setting,
-		const gchar *key_name, gint default_value, gpointer widget_id)
+		const gchar *key_name, gint default_value, GeanyWidgetID widget_id)
 {
 	add_widget_pref(group, G_TYPE_INT, setting, key_name, GINT_TO_POINTER(default_value),
 		GTK_TYPE_COMBO_BOX, widget_id);
@@ -682,7 +682,7 @@
 /* We could maybe also have something like stash_group_add_combo_box_entry_with_menu()
  * for the history list - or should that be stored as a separate setting? */
 void stash_group_add_combo_box_entry(GeanyPrefGroup *group, gchar **setting,
-		const gchar *key_name, const gchar *default_value, gpointer widget_id)
+		const gchar *key_name, const gchar *default_value, GeanyWidgetID widget_id)
 {
 	add_widget_pref(group, G_TYPE_STRING, setting, key_name, (gpointer)default_value,
 		GTK_TYPE_COMBO_BOX_ENTRY, widget_id);
@@ -690,7 +690,7 @@
 
 
 void stash_group_add_entry(GeanyPrefGroup *group, gchar **setting,
-		const gchar *key_name, const gchar *default_value, gpointer widget_id)
+		const gchar *key_name, const gchar *default_value, GeanyWidgetID widget_id)
 {
 	add_widget_pref(group, G_TYPE_STRING, setting, key_name, (gpointer)default_value,
 		GTK_TYPE_ENTRY, widget_id);
@@ -716,7 +716,7 @@
  * @warning Currently only string GValue properties will be freed before setting; patch for
  * other types - see handle_widget_property(). */
 void stash_group_add_widget_property(GeanyPrefGroup *group, gpointer setting,
-		const gchar *key_name, gpointer default_value, gpointer widget_id,
+		const gchar *key_name, gpointer default_value, GeanyWidgetID widget_id,
 		const gchar *property_name, GType type)
 {
 	if (!type)

Modified: trunk/src/stash.h
===================================================================
--- trunk/src/stash.h	2009-10-30 06:54:53 UTC (rev 4392)
+++ trunk/src/stash.h	2009-10-30 14:09:46 UTC (rev 4393)
@@ -29,7 +29,11 @@
 
 typedef struct GeanyPrefGroup GeanyPrefGroup;
 
+/* Can be (GtkWidget*) or (gchar*) depending on whether owner argument is used for
+ * stash_group_display/stash_group_update. */
+typedef gpointer GeanyWidgetID;
 
+
 GeanyPrefGroup *stash_group_new(const gchar *name);
 
 void stash_group_set_write_once(GeanyPrefGroup *group, gboolean write_once);
@@ -58,26 +62,26 @@
 /* *** GTK-related functions *** */
 
 void stash_group_add_toggle_button(GeanyPrefGroup *group, gboolean *setting,
-		const gchar *key_name, gboolean default_value, gpointer widget_id);
+		const gchar *key_name, gboolean default_value, GeanyWidgetID widget_id);
 
 void stash_group_add_radio_buttons(GeanyPrefGroup *group, gint *setting,
 		const gchar *key_name, gint default_value,
-		gpointer widget_id, gint enum_id, ...) G_GNUC_NULL_TERMINATED;
+		GeanyWidgetID widget_id, gint enum_id, ...) G_GNUC_NULL_TERMINATED;
 
 void stash_group_add_spin_button_integer(GeanyPrefGroup *group, gint *setting,
-		const gchar *key_name, gint default_value, gpointer widget_id);
+		const gchar *key_name, gint default_value, GeanyWidgetID widget_id);
 
 void stash_group_add_combo_box(GeanyPrefGroup *group, gint *setting,
-		const gchar *key_name, gint default_value, gpointer widget_id);
+		const gchar *key_name, gint default_value, GeanyWidgetID widget_id);
 
 void stash_group_add_combo_box_entry(GeanyPrefGroup *group, gchar **setting,
-		const gchar *key_name, const gchar *default_value, gpointer widget_id);
+		const gchar *key_name, const gchar *default_value, GeanyWidgetID widget_id);
 
 void stash_group_add_entry(GeanyPrefGroup *group, gchar **setting,
-		const gchar *key_name, const gchar *default_value, gpointer widget_id);
+		const gchar *key_name, const gchar *default_value, GeanyWidgetID widget_id);
 
 void stash_group_add_widget_property(GeanyPrefGroup *group, gpointer setting,
-		const gchar *key_name, gpointer default_value, gpointer widget_id,
+		const gchar *key_name, gpointer default_value, GeanyWidgetID widget_id,
 		const gchar *property_name, GType type);
 
 void stash_group_display(GeanyPrefGroup *group, GtkWidget *owner);


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



More information about the Commits mailing list