<div dir="ltr"><p dir="ltr"><br>
On Jul 18, 2013 10:55 AM, "Tanu Kaskinen" <<a href="mailto:tanu.kaskinen@linux.intel.com" target="_blank">tanu.kaskinen@linux.intel.com</a>> wrote:<br>
><br>
> On Fri, 2013-07-12 at 15:06 -0300, <a href="mailto:jprvita@gmail.com" target="_blank">jprvita@gmail.com</a> wrote:<br>
> > From: João Paulo Rechi Vita <<a href="mailto:jprvita@openbossa.org" target="_blank">jprvita@openbossa.org</a>><br>
> ><br>
> > ---<br>
> >  src/modules/bluetooth/bluez5-util.c | 101 ++++++++++++++++++++++++++++++++++++<br>
> >  1 file changed, 101 insertions(+)<br>
> ><br>
> > diff --git a/src/modules/bluetooth/bluez5-util.c b/src/modules/bluetooth/bluez5-util.c<br>
><br>
> > +static DBusHandlerResult filter_cb(DBusConnection *bus, DBusMessage *m, void *userdata) {<br>
> > +    pa_bluetooth_discovery *y;<br>
> > +    DBusError err;<br>
> > +    const char *path, *interface, *member;<br>
> > +<br>
> > +    pa_assert(bus);<br>
> > +    pa_assert(m);<br>
> > +    pa_assert_se(y = userdata);<br>
> > +<br>
> > +    path = dbus_message_get_path(m);<br>
> > +    interface = dbus_message_get_interface(m);<br>
> > +    member = dbus_message_get_member(m);<br>
> > +<br>
> > +    pa_log_debug("dbus: path=%s, interface=%s, member=%s", path, interface, member);<br>
><br>
> I don't like this message, it spams the log with all D-Bus traffic,<br>
> which does more harm than good IMO.<br>
></p>
<p dir="ltr">I personally like it, but I agree it doesn't scale well as we add more filter_cb functions (the oFono backend has a new one, for example) and that dbus-monitor should substitute this line well enough. I'll send an updated version.</p>


</div>