Hi Gustavo, On Mon, Dec 06, 2010, Gustavo F. Padovan wrote: > - return g_dbus_create_error(msg, > - ERROR_INTERFACE ".NotSupported", > - "Kernel lacks blacklist support"); > + return btd_error_not_supported(msg); I'm not sure I agree with this. The specific error message would give hints to the user that he might need to update his kernel but the generic one doesn't give any such hints. Couldn't you have the btd_error_not_supported function take one extra cont char *msg parameter? I had a little bit similar feelings about the ALREADY_EXISTS patch (which I already pushed) but here the issue of loosing the specifics of the error is more obvious. 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
