[gnome-software: 5/18] gs-app: Add G_PARAM_STATIC_STRINGS to all properties




commit 01b5c341a12be304715e4c480358a76d12588b7b
Author: Philip Withnall <pwithnall endlessos org>
Date:   Thu May 27 18:43:17 2021 +0100

    gs-app: Add G_PARAM_STATIC_STRINGS to all properties
    
    This prevents `GObject` from taking copies of the property name and
    description.
    
    Although almost all the properties don’t specify a name or description,
    it’s good to specify this anyway, in case strings are added in future,
    or the code is copy/pasted somewhere and strings are added there.
    
    Signed-off-by: Philip Withnall <pwithnall endlessos org>

 lib/gs-app.c | 34 +++++++++++++++++-----------------
 1 file changed, 17 insertions(+), 17 deletions(-)
---
diff --git a/lib/gs-app.c b/lib/gs-app.c
index aaa5d61a2..61d400d68 100644
--- a/lib/gs-app.c
+++ b/lib/gs-app.c
@@ -4871,42 +4871,42 @@ gs_app_class_init (GsAppClass *klass)
         */
        obj_props[PROP_ID] = g_param_spec_string ("id", NULL, NULL,
                                     NULL,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:name:
         */
        obj_props[PROP_NAME] = g_param_spec_string ("name", NULL, NULL,
                                     NULL,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:version:
         */
        obj_props[PROP_VERSION] = g_param_spec_string ("version", NULL, NULL,
                                     NULL,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:summary:
         */
        obj_props[PROP_SUMMARY] = g_param_spec_string ("summary", NULL, NULL,
                                     NULL,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:description:
         */
        obj_props[PROP_DESCRIPTION] = g_param_spec_string ("description", NULL, NULL,
                                     NULL,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:rating:
         */
        obj_props[PROP_RATING] = g_param_spec_int ("rating", NULL, NULL,
                                  -1, 100, -1,
-                                 G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                 G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:kind:
@@ -4916,7 +4916,7 @@ gs_app_class_init (GsAppClass *klass)
                                   AS_COMPONENT_KIND_UNKNOWN,
                                   AS_COMPONENT_KIND_LAST,
                                   AS_COMPONENT_KIND_UNKNOWN,
-                                  G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                  G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:special-kind:
@@ -4937,7 +4937,7 @@ gs_app_class_init (GsAppClass *klass)
        obj_props[PROP_STATE] = g_param_spec_enum ("state", NULL, NULL,
                                   GS_TYPE_APP_STATE,
                                   GS_APP_STATE_UNKNOWN,
-                                  G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                  G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:progress:
@@ -4949,21 +4949,21 @@ gs_app_class_init (GsAppClass *klass)
         */
        obj_props[PROP_PROGRESS] = g_param_spec_uint ("progress", NULL, NULL,
                                   0, GS_APP_PROGRESS_UNKNOWN, GS_APP_PROGRESS_UNKNOWN,
-                                  G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                  G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:allow-cancel:
         */
        obj_props[PROP_CAN_CANCEL_INSTALLATION] =
                g_param_spec_boolean ("allow-cancel", NULL, NULL, TRUE,
-                                     G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                     G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:install-date:
         */
        obj_props[PROP_INSTALL_DATE] = g_param_spec_uint64 ("install-date", NULL, NULL,
                                     0, G_MAXUINT64, 0,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:release-date:
@@ -4975,34 +4975,34 @@ gs_app_class_init (GsAppClass *klass)
         */
        obj_props[PROP_RELEASE_DATE] = g_param_spec_uint64 ("release-date", NULL, NULL,
                                     0, G_MAXUINT64, 0,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:quirk:
         */
        obj_props[PROP_QUIRK] = g_param_spec_flags ("quirk", NULL, NULL,
                                     GS_TYPE_APP_QUIRK, GS_APP_QUIRK_NONE,
-                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                    G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:pending-action:
         */
        obj_props[PROP_PENDING_ACTION] = g_param_spec_enum ("pending-action", NULL, NULL,
                                     GS_TYPE_PLUGIN_ACTION, GS_PLUGIN_ACTION_UNKNOWN,
-                                    G_PARAM_READABLE);
+                                    G_PARAM_READABLE | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:key-colors:
         */
        obj_props[PROP_KEY_COLORS] = g_param_spec_boxed ("key-colors", NULL, NULL,
-                                   G_TYPE_ARRAY, G_PARAM_READWRITE);
+                                   G_TYPE_ARRAY, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:is-update-downloaded:
         */
        obj_props[PROP_IS_UPDATE_DOWNLOADED] = g_param_spec_boolean ("is-update-downloaded", NULL, NULL,
                                               FALSE,
-                                              G_PARAM_READWRITE);
+                                              G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS);
 
        /**
         * GsApp:url-missing:
@@ -5014,7 +5014,7 @@ gs_app_class_init (GsAppClass *klass)
         */
        obj_props[PROP_URL_MISSING] = g_param_spec_string ("url-missing", NULL, NULL,
                                        NULL,
-                                       G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
+                                       G_PARAM_READWRITE | G_PARAM_CONSTRUCT | G_PARAM_STATIC_STRINGS);
 
        g_object_class_install_properties (object_class, G_N_ELEMENTS (obj_props), obj_props);
 }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]