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 750286 - Callbacks with (transfer full) return types are not transferring ownership
Callbacks with (transfer full) return types are not transferring ownership
Status: RESOLVED FIXED
Product: gjs
Classification: Bindings
Component: general
1.42.x
Other All
: Normal normal
: ---
Assigned To: gjs-maint
gjs-maint
Depends on:
Blocks: 726450
 
 
Reported: 2015-06-02 15:16 UTC by Debarshi Ray
Modified: 2016-03-21 00:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gi: function: Support callbacks with (transfer full) return types (1.29 KB, patch)
2015-06-02 15:18 UTC, Debarshi Ray
committed Details | Review
[gobject-introspection] tests: Add regression test for (transfer full) returning callbacks (1.97 KB, patch)
2015-06-02 17:28 UTC, Debarshi Ray
committed Details | Review
[gjs] tests: Add regression test for (transfer full) returning callbacks (1.08 KB, patch)
2015-06-02 17:29 UTC, Debarshi Ray
committed Details | Review

Description Debarshi Ray 2015-06-02 15:16:55 UTC
Callbacks that have (transfer full) return types are not transferring ownership, as they are supposed to. GtkListBoxCreateWidgetFunc is an example of such callbacks. This was discovered when using gtk_list_box_bind_model from gjs.

See bug 746893 for further details.
Comment 1 Debarshi Ray 2015-06-02 15:18:11 UTC
Created attachment 304441 [details] [review]
gi: function: Support callbacks with (transfer full) return types
Comment 2 Jasper St. Pierre (not reading bugmail) 2015-06-02 15:32:43 UTC
Review of attachment 304441 [details] [review]:

Looks good to me.
Comment 3 Debarshi Ray 2015-06-02 15:41:17 UTC
Comment on attachment 304441 [details] [review]
gi: function: Support callbacks with (transfer full) return types

Pushed to master.
Comment 4 Debarshi Ray 2015-06-02 17:28:53 UTC
Created attachment 304450 [details] [review]
[gobject-introspection] tests: Add regression test for (transfer full) returning callbacks
Comment 5 Debarshi Ray 2015-06-02 17:29:47 UTC
Created attachment 304451 [details] [review]
[gjs] tests: Add regression test for (transfer full) returning callbacks
Comment 6 Debarshi Ray 2015-06-18 14:41:31 UTC
From #gtk+ on GIMPNet:

14:37 <rishi> Jasper: Are these test-cases OK:                                  
      https://bugzilla.gnome.org/show_bug.cgi?id=750286 ?
14:38 <Jasper> rishi, yep.
Comment 7 Debarshi Ray 2015-06-18 14:42:43 UTC
Pushed to master.
Comment 8 Dieter Verfaillie 2015-06-19 22:28:14 UTC
(In reply to Debarshi Ray from comment #5)
> Created attachment 304451 [details] [review] [review]
> [gjs] tests: Add regression test for (transfer full) returning callbacks

Pushed missing doc-tool tests to master as commit 254ac953c6b18a0d4e93781cd9c29bf3dcd4953a
Comment 9 Michael Biebl 2016-03-21 00:04:38 UTC
Hi Debarshi,
this commit now makes the test-suite fail on various architectures.
See https://bugzilla.gnome.org/show_bug.cgi?id=757328

Can't say if this is a bug in the test itself of actually a real bug which has been uncovered by the test. In any case, would be great if you can have a look at 757328. Atm this is blocking the gjs in Debian.