On Tue, 2013-04-02 at 13:53 +0200, Daniel Wagner wrote:
> Hi Patrik
> 
> On 04/02/2013 12:34 PM, Patrik Flykt wrote:
> > +void __connmanctl_dbus_print(DBusMessageIter *iter, const char *pre,
> > +           const char *dict, const char *sep);
> > +
> > +typedef void (*connmanctl_dbus_method_return_fn)(DBusMessageIter *iter,
> > +           const char *error, void *user_data);
> 
> Nitpicking: It seems we cannot agree on the postfix on a callback 
> typedef. So far we have:
> 
> src/connman.h: typedef void (*connman_iptables_iterate_chains_cb_t)
> src/shared/util.h:typedef void (*util_debug_func_t)
> 
> oFono and BlueZ prefer the "_func_t" prefix, IIRC. So I suggest we go 
> also in this direction.

Fixed according to the nitpicking above.

Cheers,

        Patrik

_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman

Reply via email to