Hi Alecrim, A few more minor correction suggestions: On Wed, Apr 07, 2010, Francisco Alecrim wrote: > + emit_array_property_changed(connection, adapter->path, > + ADAPTER_INTERFACE, "UUIDs", > + DBUS_TYPE_STRING, &uuids); The general rule for continuation line indenting is: indent as much as possible as long as the total length doesn't go beyond 80. It looks like these two lines could be indented a bit more. There are a few other similar places too but I think you should be able to find them by yourself. > + for (i = 0; uuids[i]; i++) > + g_free(uuids[i]); > + g_free(uuids); These three lines could be replaced simply with g_strfreev(uuids); > + for (i = 0; uuids[i]; i++) > + g_free(uuids[i]); > + g_free(uuids); And 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
