Hi Andrei, >>> Function always returns 0, so there is sense to make it void function. >>> --- >>> lib/sdp.c | 6 +----- >>> lib/sdp_lib.h | 2 +- >>> 2 files changed, 2 insertions(+), 6 deletions(-) >>> >>> diff --git a/lib/sdp.c b/lib/sdp.c >>> index e5e4622..da26b78 100644 >>> --- a/lib/sdp.c >>> +++ b/lib/sdp.c >>> @@ -2380,10 +2380,8 @@ static sdp_data_t *access_proto_to_dataseq(sdp_record_t *rec, sdp_list_t *proto) >>> * >>> * This function replaces the existing sdp_access_proto_t >>> * structure (if any) with the new one specified. >>> - * >>> - * returns 0 if successful or -1 if there is a failure. >>> */ >>> -int sdp_set_access_protos(sdp_record_t *rec, const sdp_list_t *ap) >>> +void sdp_set_access_protos(sdp_record_t *rec, const sdp_list_t *ap) >> >> if this is libbluetooth API, then we can not do that. I am not planning to break that API. >> > > Is libbluetooth API internal API and only used in our code? it is a public API. Can be enabled with --enable-library in configure. Regards Marcel -- 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
