Bug 17797 - add support for 'g' type, as a boxed char *
Summary: add support for 'g' type, as a boxed char *
Status: RESOLVED FIXED
Alias: None
Product: dbus
Classification: Unclassified
Component: GLib (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Robert McQueen
QA Contact: John (J5) Palmieri
URL:
Whiteboard:
Keywords: patch
: 6069 (view as bug list)
Depends on:
Blocks: 17798
  Show dependency treegraph
 
Reported: 2008-09-26 09:02 UTC by Robert McQueen
Modified: 2011-04-14 04:34 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
add support for 'g' type, as a boxed char * (8.64 KB, patch)
2008-09-26 09:10 UTC, Robert McQueen
Details | Splinter Review

Description Robert McQueen 2008-09-26 09:02:34 UTC
Adds DBUS_G_TYPE_SIGNATURE, which is a boxed type just like
DBUS_G_TYPE_OBJECT_PATH.
Comment 1 Robert McQueen 2008-09-26 09:10:28 UTC
Created attachment 19244 [details] [review]
add support for 'g' type, as a boxed char *
Comment 2 Colin Walters 2008-09-30 08:42:19 UTC
Instead of:
+  g_value_set_boxed_take_ownership (value, g_strdup (sig));

You can just write:

g_value_set_boxed (value, sig);

which seems like less of a dance.

Otherwise looks good, please apply.
Comment 3 Robert McQueen 2008-11-28 19:23:52 UTC
Fixed _take_ownership/g_strdup in this and the boxed object path where I copy/pasted the code from. Merged. :)
Comment 4 Simon McVittie 2011-04-14 04:34:49 UTC
*** Bug 6069 has been marked as a duplicate of this bug. ***


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.