Re: [PATCH] Use g_cclosure_marshal_generic



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]