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 647477 - [gi] Added support for GVariant arguments
[gi] Added support for GVariant arguments
Status: RESOLVED FIXED
Product: pygobject
Classification: Bindings
Component: introspection
unspecified
Other All
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on:
Blocks:
 
 
Reported: 2011-04-11 16:42 UTC by Sebastian Pölsterl
Modified: 2011-04-11 17:22 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
[gi] Added support for GVariant arguments (1.14 KB, patch)
2011-04-11 16:42 UTC, Sebastian Pölsterl 		committed Details | Review

Description Sebastian Pölsterl 2011-04-11 16:42:50 UTC 
Currently, you will get an assertion warning from g_value_get_pointer.
This patch fixes this by adding a case for GValue containing a GVariant.
Comment 1 Sebastian Pölsterl 2011-04-11 16:42:53 UTC 
Created attachment 185730 [details] [review]
[gi] Added support for GVariant arguments

This is required in order for the "g-signal" signal of GDBusProxy to work properly and thus to properly receive DBus signals with any type of argument.
Comment 2 Tomeu Vizoso 2011-04-11 16:48:29 UTC 
Review of attachment 185730 [details] [review]:

Looks great :)
Comment 3 Sebastian Pölsterl 2011-04-11 17:21:52 UTC 
Committed to master and pygobject-2-28 branch.