Hi Waldek, On Wed, Nov 03, 2010, Waldemar Rymarkiewicz wrote: > +++ b/sap/main.c > @@ -0,0 +1,54 @@ > +/* > + * BlueZ - Bluetooth protocol stack for Linux > + * > + * Copyright (C) 2010 Claudio Takahasi<claudio.takahasi@xxxxxxxxxxxxx> As discussed off-list, this needs fixing. > +++ b/sap/manager.c > @@ -0,0 +1,94 @@ > +/* > + * BlueZ - Bluetooth protocol stack for Linux > + * > + * Copyright (C) 2010 Claudio Takahasi<claudio.takahasi@xxxxxxxxxxxxx> Same here. > +static DBusConnection *connection = NULL; > + > + > +static int sap_server_probe(struct btd_adapter *adapter) There should never be the need to have two consecutive empty lines. Please remove one. > + const gchar *path = adapter_get_path(adapter); Please avoid using the glib specific types when standard C types can be used. I.e. use just "char" here. > +static void sap_server_remove(struct btd_adapter *adapter) > +{ > + const gchar *path = adapter_get_path(adapter); Same here. > +++ b/sap/manager.h > @@ -0,0 +1,21 @@ > +/* > + * BlueZ - Bluetooth protocol stack for Linux > + * > + * Copyright (C) 2010 Claudio Takahasi<claudio.takahasi@xxxxxxxxxxxxx> Same copyright fix needed as previously discussed. > +++ b/sap/server.h > @@ -0,0 +1,25 @@ > +/* > + * BlueZ - Bluetooth protocol stack for Linux > + * > + * Copyright (C) 2010 Claudio Takahasi<claudio.takahasi@xxxxxxxxxxxxx> Same here. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html