Re: [PATCH] Use g_cclosure_marshal_generic
- From: Colin Walters <walters verbum org>
- To: Dan Winship <danw gnome org>
- Cc: networkmanager-list gnome org
- Subject: Re: [PATCH] Use g_cclosure_marshal_generic
- Date: Wed, 08 May 2013 09:42:58 -0400
On Wed, 2013-05-08 at 09:22 -0400, Dan Winship wrote:
Oh, I just did this too as part of the danw/movedevs branch...
- dbus_g_object_register_marshaller (_nm_marshal_VOID__STRING_BOXED,
+ dbus_g_object_register_marshaller (NULL,
Does that work? From what I can see in the dbus-glib source code, it
seems like you would need to pass g_cclosure_marshal_generic explicitly,
not NULL.
My read of the source agrees; I suspect I simply didn't hit any of these
in my testing (boot gnome-ostree buildmaster in a VM).
I'll update the patch.
But if someone reviews my dbus-glib patch, that'd be awesome, so we can
eventually depend on it.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]