Created attachment 16763 [details] [review] uscore property names for GetAll() too Everywhere else converts the property names, but GetAll() doesn't seem to. Results in GetAll() returning no properties, becuase the properties from the introspection data are WinCaps, but the gobject properties are uscored.
Created attachment 16764 [details] [review] new version that doesn't use-after-free in the error case
Sounds sensible (didn't look at the patch though) to me, I didn't think of that when writing the original GetAll() patch.
So this isn't an API/ABI change in any way, it was basically that the old GetAll didn't work?
(In reply to comment #3) > So this isn't an API/ABI change in any way, it was basically that the old > GetAll didn't work? Sort of. But keep in mind that the patch for GetAll() was never in any release.
Ok, sounds good then; David can you commit?
This is an ABI change, but if you were using dbus-glib such that this patch affects you, then Get/Set wouldn't have worked for you already either. Get/Set operations convert the WinCaps property name to a u-scored name, so if your GObject properties were WinCaps already (in which case this patch would break you) then the Get/Set would fail for your object because the g_object_class_find_property() for the WinCaps->u-scored name would already fail.
commit d1b80d803a0268bd4b3dd5b9a9522230461f2947 Author: Dan Williams <dcbw@redhat.com> Date: Thu Jun 5 17:57:53 2008 -0400 Bug 16114 [patch] wincaps-to-uscore property names for GetAll() * dbus/dbus-gobject.c: We need to uscore property names so that we actually find the right properties.
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.