Hi Antti, 2010/12/3 Antti Paila <antti.pa...@nokia.com>: > +static GDBusMethodTable nettime_methods[] = { > + { "GetNetworkTime", "", "a{sv}", nettime_get_time }, > + { } > +}; > + > +static GDBusSignalTable nettime_signals[] = { > + { "NetworkTimeChanged", "a{sv}" }, > + { } > +};
The only thing I would change here are the names. It's customary to avoid repeating the interface name in the signal or method, but of course the problem here is then what to call these. Maybe just Get() and Changed()? ;-) Oh, and you probably should have included a short description of what this plugin is for (I forgot to mention this before). The thinking behind this plugin is that timed in MeeGo wants to get the information, and a D-Bus API is a fine way to accomplish this. Cheers, Aki _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono