Nicolas Morey-Chaisemartin <[email protected]> writes:
>
> + if (cred.username)
> + if (res == CURLE_OK)
> + credential_approve(&cred);
> +#if LIBCURL_VERSION_NUM >= 0x070d01
> + else if (res == CURLE_LOGIN_DENIED)
> +#else
> + else
> +#endif
> + credential_reject(&cred);
> +
> + credential_clear(&cred);
> +
As my copy of GCC seemed to be worried about readers getting
confused by the if/else cascade, I'd place an extra pair of braces
around this, i.e.
if (cred.username) {
if (res == CURLE_OK)
credential_approve(&cred);
else /* or "else if DENIED" */
credential_reject(&cred);
}
credential_clear(&cred);
while queuing this patch.
