Re: [PATCH v2 1/2] shared/hfp: Remove reduntant check

[Szymon Janc <szymon.janc@tieto.com>]

Hi Łukasz,

On Friday 31 of October 2014 10:32:32 Lukasz Rymanowski wrote:
> To make this check strcmp is good enought. Not need to check len first
> and then do memory compare.
> ---
>  src/shared/hfp.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/src/shared/hfp.c b/src/shared/hfp.c
> index 9a66ed2..0ce38cc 100644
> --- a/src/shared/hfp.c
> +++ b/src/shared/hfp.c
> @@ -135,10 +135,7 @@ 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)))
> +	if (strcmp(handler->prefix, prefix) != 0)
>  		return false;
> 
>  	return true;

Applied (with typos in commit message fixed), thanks.

-- 
BR
Szymon Janc