From 61912bffb59140416e1a55461230db847d448740 Mon Sep 17 00:00:00 2001 From: Philip Withnall Date: Thu, 29 Aug 2019 09:12:18 +0100 Subject: [PATCH] tutorial: Improve type safety of property usage in GObject tutorial MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This means that if you compile with `-Wswitch-enum`, the compiler will warn you about properties which you’ve forgotten to handle in `set_property()` or `get_property()`. Signed-off-by: Philip Withnall Helps: #1858 --- docs/reference/gobject/tut_gobject.xml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/reference/gobject/tut_gobject.xml b/docs/reference/gobject/tut_gobject.xml index 161150ebf..e6d7923f5 100644 --- a/docs/reference/gobject/tut_gobject.xml +++ b/docs/reference/gobject/tut_gobject.xml @@ -465,12 +465,12 @@ ViewerFile *file = g_object_new (VIEWER_TYPE_FILE, NULL); /* Implementation */ /************************************************/ -enum +typedef enum { PROP_FILENAME = 1, PROP_ZOOM_LEVEL, N_PROPERTIES -}; +} ViewerFileProperty; static GParamSpec *obj_properties[N_PROPERTIES] = { NULL, }; @@ -482,7 +482,7 @@ viewer_file_set_property (GObject *object, { ViewerFile *self = VIEWER_FILE (object); - switch (property_id) + switch ((ViewerFileProperty) property_id) { case PROP_FILENAME: g_free (self->priv->filename); @@ -510,7 +510,7 @@ viewer_file_get_property (GObject *object, { ViewerFile *self = VIEWER_FILE (object); - switch (property_id) + switch ((ViewerFileProperty) property_id) { case PROP_FILENAME: g_value_set_string (value, self->priv->filename);