diff options
| author | Ryan Lortie <desrt@desrt.ca> | 2012-11-19 10:28:55 -0500 |
|---|---|---|
| committer | Ryan Lortie <desrt@desrt.ca> | 2012-11-19 10:30:47 -0500 |
| commit | 5272bc4d67ff6c100ba02b02869db80f51d37c26 (patch) | |
| tree | c370ca27f4a5c9ac7e3e7e67f223b0ff5ddf5b3c | |
| parent | 39150f90e5c34e5402c1a31ed9e71d2e21cfd4c0 (diff) | |
| download | glib-5272bc4d67ff6c100ba02b02869db80f51d37c26.tar.gz | |
Revert "[gobject] set all properties before constructed()"
This reverts commit 028d4a03f2baa7ca9243a2819f30ffb0d5e137d7.
I thought that we would be able to get away with this incompatible
change but it appears to impact far too much existing code. The only
thing we can do is revert.
https://bugzilla.gnome.org/show_bug.cgi?id=688596
| -rw-r--r-- | gobject/gobject.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gobject/gobject.c b/gobject/gobject.c index 5f60dd6da..79c04011b 100644 --- a/gobject/gobject.c +++ b/gobject/gobject.c @@ -1741,15 +1741,15 @@ g_object_newv (GType object_type, g_object_notify_queue_thaw (object, nqueue); } + /* run 'constructed' handler if there is a custom one */ + if (newly_constructed && CLASS_HAS_CUSTOM_CONSTRUCTED (class)) + class->constructed (object); + /* set remaining properties */ for (i = 0; i < n_oparams; i++) object_set_property (object, oparams[i].pspec, oparams[i].value, nqueue); g_free (oparams); - /* run 'constructed' handler if there is a custom one */ - if (newly_constructed && CLASS_HAS_CUSTOM_CONSTRUCTED (class)) - class->constructed (object); - if (CLASS_HAS_PROPS (class)) { /* release our own freeze count and handle notifications */ |