Re: wldap32: Use BOOL type where appropriate

[Rico Schüller]

On 01.10.2013 00:06, Frédéric 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