[devhelp] Settings lib: rename "singleton" -> "default instance"
- From: Sébastien Wilmet <swilmet src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [devhelp] Settings lib: rename "singleton" -> "default instance"
- Date: Mon, 2 Apr 2018 06:16:14 +0000 (UTC)
commit fad7d685f278670527b819edcdae9b7a4b585329
Author: Sébastien Wilmet <swilmet gnome org>
Date: Mon Apr 2 08:07:55 2018 +0200
Settings lib: rename "singleton" -> "default instance"
Another step towards being able to customize the GSettings paths in an
app.
devhelp/dh-book-manager.c | 6 +++---
devhelp/dh-init.c | 2 +-
devhelp/dh-settings.c | 25 ++++++++++++-------------
devhelp/dh-settings.h | 4 ++--
src/dh-preferences.c | 2 +-
5 files changed, 19 insertions(+), 20 deletions(-)
---
diff --git a/devhelp/dh-book-manager.c b/devhelp/dh-book-manager.c
index 98d3f94..c326cbd 100644
--- a/devhelp/dh-book-manager.c
+++ b/devhelp/dh-book-manager.c
@@ -355,7 +355,7 @@ load_books_disabled (DhBookManager *book_manager)
g_assert (priv->books_disabled == NULL);
- settings = dh_settings_get_singleton ();
+ settings = dh_settings_get_default ();
contents_settings = dh_settings_peek_contents_settings (settings);
books_disabled_strv = g_settings_get_strv (contents_settings, "books-disabled");
@@ -392,7 +392,7 @@ store_books_disabled (DhBookManager *book_manager)
variant = g_variant_builder_end (builder);
g_variant_builder_unref (builder);
- settings = dh_settings_get_singleton ();
+ settings = dh_settings_get_default ();
contents_settings = dh_settings_peek_contents_settings (settings);
g_settings_set_value (contents_settings, "books-disabled", variant);
}
@@ -837,7 +837,7 @@ dh_book_manager_init (DhBookManager *book_manager)
load_books_disabled (book_manager);
- settings = dh_settings_get_singleton ();
+ settings = dh_settings_get_default ();
contents_settings = dh_settings_peek_contents_settings (settings);
g_settings_bind (contents_settings, "group-books-by-language",
book_manager, "group-by-language",
diff --git a/devhelp/dh-init.c b/devhelp/dh-init.c
index 30f45e4..bdc8b11 100644
--- a/devhelp/dh-init.c
+++ b/devhelp/dh-init.c
@@ -82,7 +82,7 @@ dh_finalize (void)
*/
if (!done) {
_dh_book_manager_unref_singleton ();
- _dh_settings_unref_singleton ();
+ _dh_settings_unref_default ();
done = TRUE;
}
}
diff --git a/devhelp/dh-settings.c b/devhelp/dh-settings.c
index a23ce4c..09dfb7c 100644
--- a/devhelp/dh-settings.c
+++ b/devhelp/dh-settings.c
@@ -28,8 +28,7 @@ struct _DhSettingsPrivate {
GSettings *settings_contents;
};
-/* DhSettings is a singleton. */
-static DhSettings *singleton = NULL;
+static DhSettings *default_instance = NULL;
G_DEFINE_TYPE_WITH_PRIVATE (DhSettings, dh_settings, G_TYPE_OBJECT);
@@ -46,8 +45,8 @@ dh_settings_dispose (GObject *object)
static void
dh_settings_finalize (GObject *object)
{
- if (singleton == DH_SETTINGS (object))
- singleton = NULL;
+ if (default_instance == DH_SETTINGS (object))
+ default_instance = NULL;
G_OBJECT_CLASS (dh_settings_parent_class)->finalize (object);
}
@@ -91,23 +90,23 @@ _dh_settings_new (const gchar *contents_path)
}
DhSettings *
-dh_settings_get_singleton (void)
+dh_settings_get_default (void)
{
- if (singleton == NULL) {
- singleton = _dh_settings_new (/* Must be compatible with Devhelp app version 3.28: */
- "/org/gnome/devhelp/state/main/contents/");
+ if (default_instance == NULL) {
+ default_instance = _dh_settings_new (/* Must be compatible with Devhelp app version 3.28: */
+ "/org/gnome/devhelp/state/main/contents/");
}
- return singleton;
+ return default_instance;
}
void
-_dh_settings_unref_singleton (void)
+_dh_settings_unref_default (void)
{
- if (singleton != NULL)
- g_object_unref (singleton);
+ if (default_instance != NULL)
+ g_object_unref (default_instance);
- /* singleton is not set to NULL here, it is set to NULL in
+ /* default_instance is not set to NULL here, it is set to NULL in
* dh_settings_finalize() (i.e. when we are sure that the ref count
* reaches 0).
*/
diff --git a/devhelp/dh-settings.h b/devhelp/dh-settings.h
index 95645f2..009dd3d 100644
--- a/devhelp/dh-settings.h
+++ b/devhelp/dh-settings.h
@@ -51,10 +51,10 @@ struct _DhSettingsClass {
GType dh_settings_get_type (void) G_GNUC_CONST;
-DhSettings * dh_settings_get_singleton (void);
+DhSettings * dh_settings_get_default (void);
G_GNUC_INTERNAL
-void _dh_settings_unref_singleton (void);
+void _dh_settings_unref_default (void);
GSettings * dh_settings_peek_contents_settings (DhSettings *self);
diff --git a/src/dh-preferences.c b/src/dh-preferences.c
index bbeadf6..2d1b59f 100644
--- a/src/dh-preferences.c
+++ b/src/dh-preferences.c
@@ -622,7 +622,7 @@ dh_preferences_init (DhPreferences *prefs)
/* setup GSettings bindings */
settings_app = dh_settings_app_get_singleton ();
settings_fonts = dh_settings_app_peek_fonts_settings (settings_app);
- settings_lib = dh_settings_get_singleton ();
+ settings_lib = dh_settings_get_default ();
settings_contents = dh_settings_peek_contents_settings (settings_lib);
g_settings_bind (settings_fonts, "use-system-fonts",
priv->system_fonts_button, "active",
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]