[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: wldap32: Use BOOL type where appropriate
From:       Rico_Schüller <kgbricola () web ! de>
Date:       2013-09-30 22:20:58
Message-ID: 5249F94A.7030208 () web ! de
[Download RAW message or body]

On 01.10.2013 00:06, Fr=E9d=E9ric Delanoy wrote:
> ---
>   dlls/wldap32/init.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c
> index b5aed58..54b5b24 100644
> --- a/dlls/wldap32/init.c
> +++ b/dlls/wldap32/init.c
> @@ -113,13 +113,13 @@ oom:
>   }
>
>   /* Determine if a URL starts with a known LDAP scheme */
> -static int has_ldap_scheme( char *url )
> +static BOOL has_ldap_scheme( char *url )
>   {
>       if (!strncasecmp( url, "ldap://", 7 ) ||
>           !strncasecmp( url, "ldaps://", 8 ) ||
>           !strncasecmp( url, "ldapi://", 8 ) ||
> -        !strncasecmp( url, "cldap://", 8 )) return 1;
> -    return 0;
> +        !strncasecmp( url, "cldap://", 8 )) return TRUE;
> +    return FALSE;
>   }
>
>   /* Flatten an array of hostnames into a space separated string of URLs.
>
You probably may return the "value" of the if statement directly instead =

of returning TRUE / FALSE.

Cheers
Rico


[prev in list] [next in list] [prev in thread] [next in thread]