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 697020 - Fails to marshal out arrays
Fails to marshal out arrays
Status: RESOLVED FIXED
Product: gjs
Classification: Bindings
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gjs-maint
gjs-maint
Depends on:
Blocks:
 
 
Reported: 2013-04-01 08:09 UTC by Paolo Borelli
Modified: 2017-01-06 06:02 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
tests: add array_out_param unit test (1.74 KB, patch)
2013-04-01 08:09 UTC, Paolo Borelli 		committed Details | Review

Description Paolo Borelli 2013-04-01 08:09:55 UTC 
This patch adds a currently failing unit test for a vfunc with a array output param. The corresponding part is already present in g-i.
I think this (one of) the reason GApplication::local_command_line is not handled correctly.
Comment 1 Paolo Borelli 2013-04-01 08:09:57 UTC 
Created attachment 240278 [details] [review]
tests: add array_out_param unit test
Comment 2 Cosimo Cecchi 2015-10-28 01:00:14 UTC 
Paolo, I don't see callback_array_out_parameter() in GIMarshallingTests inside gobject-introspection (nor in the git log), but you mentioned the g-i part being already there.
Anything I'm missing? Is this still relevant?
Comment 3 Paolo Borelli 2015-10-28 07:32:45 UTC 
It's been a while, but after a grep I think it is vfunc_array_out_parameter in g-i
Comment 4 Philip Chimento 2017-01-06 06:00:20 UTC 
callback_array_out_parameter() indeed doesn't exist yet, but vfunc_array_out_parameter() does.

I believe the requested functionality has been implemented in the meantime, so I'm pushing the patch, with the callback_... test marked pending.
Comment 5 Philip Chimento 2017-01-06 06:01:57 UTC 
Attachment 240278 [details] pushed as ea1fc16 - tests: add array_out_param unit test