[gtk/fix-builder-parsing] builder: Fix parsing of mixed declarations

[Matthias Clasen <matthiasc src gnome org>]

commit 223c1666474b14b0c5ea910e4eeb72a357eb9f50
Author: Matthias Clasen <mclasen redhat com>
Date:   Fri Sep 24 16:34:48 2021 -0400

    builder: Fix parsing of mixed declarations
    
    The GtkBuilder parser constructs the object e.g.
    when handling a <binding> element. There may be
    more <property> elements after it, which we were
    just not applying. Fix that by always applying
    property when we see </object>. To do that, we
    need to track the applied status per property.
    
    Fixes: #4208

 gtk/gtkbuilder.c        | 10 ++++++++--
 gtk/gtkbuilderparser.c  |  5 -----
 gtk/gtkbuilderprivate.h |  6 +++---
 3 files changed, 11 insertions(+), 10 deletions(-)
---
diff --git a/gtk/gtkbuilder.c b/gtk/gtkbuilder.c
index 809f55ff63..02b4bfca41 100644
--- a/gtk/gtkbuilder.c
+++ b/gtk/gtkbuilder.c
@@ -540,6 +540,9 @@ gtk_builder_get_parameters (GtkBuilder         *builder,
       const char *property_name = prop->pspec->name;
       GValue property_value = G_VALUE_INIT;
 
+      if (prop->applied)
+        continue;
+
       if (prop->value)
         {
           g_value_init (&property_value, G_PARAM_SPEC_VALUE_TYPE (prop->pspec));
@@ -567,6 +570,8 @@ gtk_builder_get_parameters (GtkBuilder         *builder,
           GObject *object = g_hash_table_lookup (priv->objects,
                                                  g_strstrip (prop->text->str));
 
+          prop->applied = TRUE;
+
           if (object)
             {
               g_value_init (&property_value, G_OBJECT_TYPE (object));
@@ -588,8 +593,7 @@ gtk_builder_get_parameters (GtkBuilder         *builder,
               property->value = g_strdup (prop->text->str);
               property->line = prop->line;
               property->col = prop->col;
-              priv->delayed_properties = g_slist_prepend (priv->delayed_properties,
-                                                          property);
+              priv->delayed_properties = g_slist_prepend (priv->delayed_properties, property);
               continue;
             }
         }
@@ -606,6 +610,8 @@ gtk_builder_get_parameters (GtkBuilder         *builder,
           continue;
         }
 
+      prop->applied = TRUE;
+
       if (prop->pspec->flags & filter_flags)
         {
           if (filtered_parameters)
diff --git a/gtk/gtkbuilderparser.c b/gtk/gtkbuilderparser.c
index 0e614ccdac..c17062849d 100644
--- a/gtk/gtkbuilderparser.c
+++ b/gtk/gtkbuilderparser.c
@@ -470,9 +470,6 @@ builder_construct (ParserData  *data,
 
   g_assert (object_info != NULL);
 
-  if (object_info->object && object_info->applied_properties)
-    return object_info->object;
-
   if (object_info->object == NULL)
     {
       object = _gtk_builder_construct (data->builder, object_info, error);
@@ -488,8 +485,6 @@ builder_construct (ParserData  *data,
       _gtk_builder_apply_properties (data->builder, object_info, error);
     }
 
-  object_info->applied_properties = TRUE;
-
   g_assert (G_IS_OBJECT (object));
 
   object_info->object = object;
diff --git a/gtk/gtkbuilderprivate.h b/gtk/gtkbuilderprivate.h
index 913fa263a9..32cb470f4b 100644
--- a/gtk/gtkbuilderprivate.h
+++ b/gtk/gtkbuilderprivate.h
@@ -53,7 +53,6 @@ typedef struct {
 
   GObject *object;
   CommonInfo *parent;
-  gboolean applied_properties;
 } ObjectInfo;
 
 typedef struct {
@@ -71,8 +70,9 @@ typedef struct {
   GParamSpec *pspec;
   gpointer value;
   GString *text;
-  gboolean translatable:1;
-  gboolean bound:1;
+  gboolean translatable : 1;
+  gboolean bound        : 1;
+  gboolean applied      : 1;
   char *context;
   int line;
   int col;