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 654955 - improve GVariant support for object path lists
improve GVariant support for object path lists
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gvariant
unspecified
Other All
: Normal normal
: ---
Assigned To: Allison Karlitskaya (desrt)
gtkdev
Depends on:
Blocks:
 
 
Reported: 2011-07-20 10:37 UTC by Allison Karlitskaya (desrt)
Modified: 2011-07-21 13:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
GVariant: better support for object path arrays (15.41 KB, patch)
2011-07-20 11:17 UTC, Allison Karlitskaya (desrt)
none Details | Review

Description Allison Karlitskaya (desrt) 2011-07-20 10:37:10 UTC
The D-Bus type "ao" frequently appears in D-Bus services but is not well-supported in GVariant.
Comment 1 Allison Karlitskaya (desrt) 2011-07-20 11:17:31 UTC
Created attachment 192295 [details] [review]
GVariant: better support for object path arrays

Add G_VARIANT_TYPE_OBJECT_PATH_ARRAY along with accessor functions
g_variant_new_objv, g_variant_get_objv and g_variant_dup_objv.  Also add
support for '^ao' and '^a&o' format strings for g_variant_new() and
g_variant_get().
Comment 2 David Zeuthen (not reading bugmail) 2011-07-20 13:55:08 UTC
Sounds good to me. It's good timing too, because adding this to gdbus-codegen(1) will break the ABI guarantees on generated code (we can break it now, but not after 2.30...)
Comment 3 Allison Karlitskaya (desrt) 2011-07-20 14:28:10 UTC
Pushed.
Comment 4 David Zeuthen (not reading bugmail) 2011-07-21 13:35:00 UTC
I just committed the gdbus-codegen changes, see

 http://git.gnome.org/browse/glib/commit/?id=419bfe325c40604268dbb7fa050e355c9681577e