GNOME Bugzilla – Bug 731976
Use a gdbus-codegen-ed GDBusInterfaceSkeleton and GApplication
Last modified: 2014-06-24 15:37:42 UTC
... instead of manually handling method calls, properties, automatic timeouts, etc.. While going through the patch to add an extra 'as' parameter to RefreshDB (bug 728878), I realized that we don't need to deal with so much detail about acquiring the bus, registering objects on it, unwrapping the parameters from the GVariant, etc.. This is only going to get more complicated as the API evolves (eg., if we add a signal in the future). I think that using a gdbus-codegen-ed GDBusInterfaceSkeleton and GApplication makes things easier by reducing some of these details and generating the corresponding C API from the XML for us. If not anything else, it makes it more consistent with all the code out there that is already using GApplication and gdbus-codegen.
Created attachment 278850 [details] [review] Use a gdbus-codegen-ed GDBusInterfaceSkeleton and GApplication
Created attachment 278851 [details] [review] Remove unused MINER_OBJECT_PATH
I am pushing these now, so that I can rebase the patches in bug 728878 on top of these.
Created attachment 279125 [details] [review] Use a gdbus-codegen-ed GDBusInterfaceSkeleton and GApplication Fix make distcheck.