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)
A slight tangent. This is in line with the way in which we do
conditional compilation to work with different versions of libCurl,
but we recently had discussion on modernizing these version based
conditional compilation to use feature based one in another topic.
We may want to switch to
#if defined(CURLE_LOGIN_DENIED)
...
(cf.
https://public-inbox.org/git/[email protected]/
the entire thread).
No need to change _this_ patch in this series, but something to keep
in mind planning for a future follow-up work to clean things up.
Thanks.
