Bug 19927 - dbus_g_method_return_error should accept const GError
Summary: dbus_g_method_return_error should accept const GError
Status: VERIFIED FIXED
Alias: None
Product: dbus
Classification: Unclassified
Component: GLib (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Rob Taylor
QA Contact: John (J5) Palmieri
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2009-02-03 03:12 UTC by Alberto Mardegan
Modified: 2009-04-16 22:41 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

Attachments
0001-Bug-19927-Use-const-for-GError-param-we-re-not-m.patch (1.52 KB, patch)
2009-02-25 08:29 UTC, Colin Walters 		Details | Splinter Review
View All

Description Alberto Mardegan 2009-02-03 03:12:15 UTC 
dbus_g_method_return_error is not modifying or freeing the GError it receives, so its error parameter should have the "const" modifier added.
Comment 1 Colin Walters 2009-02-25 08:29:45 UTC 
Created attachment 23286 [details] [review]
0001-Bug-19927-Use-const-for-GError-param-we-re-not-m.patch
Comment 2 Simon McVittie 2009-04-15 13:11:05 UTC 
Colin: not that I'm really a reviewer for dbus-glib, but I approve of this patch. (Any chance you could look at some of mine? :-)
Comment 3 Colin Walters 2009-04-16 10:03:05 UTC 
Thanks for the review, pushed.

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.