Re: [PATCH 3/9] Add btd_error_not_supported()

Johan Hedberg
2010-12-07T21:03:39Z

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