Hi Łukasz, On Thursday 30 of October 2014 00:16:22 Lukasz Rymanowski wrote: > This check is not needed. Below memcmp check is sufficient > --- > src/shared/hfp.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/src/shared/hfp.c b/src/shared/hfp.c > index 22e9622..f5a812d 100644 > --- a/src/shared/hfp.c > +++ b/src/shared/hfp.c > @@ -135,9 +135,6 @@ static bool match_handler_prefix(const void *a, const > void *b) const struct cmd_handler *handler = a; > const char *prefix = b; > > - if (strlen(handler->prefix) != strlen(prefix)) > - return false; > - > if (memcmp(handler->prefix, prefix, strlen(prefix))) > return false; So, why not just use strcmp here instead? -- BR Szymon Janc -- 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
