After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 686697 - gtk_clutter_texture_new() can't be used from gir
gtk_clutter_texture_new() can't be used from gir
Status: RESOLVED FIXED
Product: clutter-gtk
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: clutter-gtk maintainer(s)
clutter-gtk maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-10-23 10:58 UTC by Guillaume Desmottes
Modified: 2012-10-24 09:35 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
python test script (125 bytes, text/x-python)
2012-10-23 10:58 UTC, Guillaume Desmottes 		  Details
remove gtk_clutter_texture_new() annotation (899 bytes, patch)
2012-10-24 09:13 UTC, Guillaume Desmottes 		committed Details | Review

Description Guillaume Desmottes 2012-10-23 10:58:12 UTC 
Created attachment 227049 [details]
python test script

The attached simple Python test script segfaults and a similar gjs script does as well.

The newly created texture is destroyed right away and so we try to print a finalized object;

Removing the "(transfer full)" annotation from gtk_clutter_texture_new() solves the crash but I'm not sure why as looks fine to me.
Comment 1 Guillaume Desmottes 2012-10-24 09:13:07 UTC 
I checked with Tomeu and according to him constructor are not supposed to have any annotation so that's probably the right fix.
Comment 2 Guillaume Desmottes 2012-10-24 09:13:16 UTC 
Created attachment 227126 [details] [review]
remove gtk_clutter_texture_new() annotation

Constructor should not have any annotation.
Comment 3 Emmanuele Bassi (:ebassi) 2012-10-24 09:29:03 UTC 
Review of attachment 227126 [details] [review]:

I think it's more a general case of "initially unowned is not a transfer type"; it's fine by me to remove the transfer annotation - though you probably want to go through the rest of the constructors as well.
Comment 4 Guillaume Desmottes 2012-10-24 09:33:20 UTC 
Looks like this was the only constructor annotated in clutter-gtk so we should be good there. Thanks for the review, I'll merge my patch.
Comment 5 Guillaume Desmottes 2012-10-24 09:33:48 UTC 
Attachment 227126 [details] pushed as 91b8cd2 - remove gtk_clutter_texture_new() annotation
Comment 6 Guillaume Desmottes 2012-10-24 09:35:27 UTC 
I merged to the 1.12 branch as well.
Comment 7 Guillaume Desmottes 2012-10-24 09:35:43 UTC 
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.