schema source: avoid introspection confusion
authorJasper St. Pierre <jstpierre@mecheye.net>
Wed, 16 Nov 2011 19:53:03 +0000 (14:53 -0500)
committerRyan Lortie <desrt@desrt.ca>
Thu, 17 Nov 2011 14:03:39 +0000 (14:03 +0000)
Any method that has its prefix'd argument as its first parameter will be
interpreted by introspection as a method. We don't want this, so we need
to swap the first two parameters.

gio/gsettingsschema.c
gio/gsettingsschema.h
gio/tests/gsettings.c

index 235349a..95aeca4 100644 (file)
@@ -95,8 +95,8 @@ g_settings_schema_source_unref (GSettingsSchemaSource *source)
 }
 
 GSettingsSchemaSource *
-g_settings_schema_source_new_from_directory (GSettingsSchemaSource  *parent,
-                                             const gchar            *directory,
+g_settings_schema_source_new_from_directory (const gchar            *directory,
+                                             GSettingsSchemaSource  *parent,
                                              gboolean                trusted,
                                              GError                **error)
 {
index b3c2cd5..b6974df 100644 (file)
@@ -33,8 +33,8 @@ GSettingsSchemaSource * g_settings_schema_source_get_default            (void);
 GSettingsSchemaSource * g_settings_schema_source_ref                    (GSettingsSchemaSource  *source);
 void                    g_settings_schema_source_unref                  (GSettingsSchemaSource  *source);
 
-GSettingsSchemaSource * g_settings_schema_source_new_from_directory     (GSettingsSchemaSource  *parent,
-                                                                         const gchar            *directory,
+GSettingsSchemaSource * g_settings_schema_source_new_from_directory     (const gchar            *directory,
+                                                                         GSettingsSchemaSource  *parent,
                                                                          gboolean                trusted,
                                                                          GError                **error);
 
index 968a5ee..bb21bcd 100644 (file)
@@ -1903,12 +1903,12 @@ test_schema_source (void)
 
   /* make sure it fails properly */
   parent = g_settings_schema_source_get_default ();
-  source = g_settings_schema_source_new_from_directory (parent, "/path/that/does/not/exist", TRUE, &error);
+  source = g_settings_schema_source_new_from_directory ("/path/that/does/not/exist", parent,  TRUE, &error);
   g_assert (source == NULL);
   g_clear_error (&error);
 
   /* create a source with the parent */
-  source = g_settings_schema_source_new_from_directory (parent, "schema-source", TRUE, &error);
+  source = g_settings_schema_source_new_from_directory ("schema-source", parent, TRUE, &error);
   g_assert_no_error (error);
   g_assert (source != NULL);
 
@@ -1949,7 +1949,7 @@ test_schema_source (void)
   g_settings_schema_source_unref (source);
 
   /* try again, but with no parent */
-  source = g_settings_schema_source_new_from_directory (NULL, "schema-source", FALSE, NULL);
+  source = g_settings_schema_source_new_from_directory ("schema-source", NULL, FALSE, NULL);
   g_assert (source != NULL);
 
   /* should not find it this time, even if recursive... */