On Sun, 11 Sep 2022, Patrick Monnerat wrote:
It depends on what you mean. It is solved for all systems that have a 64
bit size variable type.
I don't think it is: I did not check everywhere, but at least Curl_handler
protocol and family fields are still unsigned ints.
Oh right. But also out of a coincidence we haven't yet had to check for a bit
over 31 bits - and when we go there we'll notice we need to bump the size of
that struct field (as we get compiler errors otherwise).
I am not sure if there actually still are systems in use that don't.
That's why I was suggesting to constrain curl_off_t >= 64 bit.
So far we can just limit new protocols for curl_off_t >= 64 bit systems. If
nobody shows up wanting those protocols on systems without 64 bit types, then
we don't have to address that issue.
I'm in favor of using a protocol number enum for single protocol values and
an FD_SET-like strategy for allowed_protocols and redir_protocols.
Yeah, I just don't think we need to go there yet.
BTW: we'll soon have the same problem with feature bits.
Yeps.
--
/ daniel.haxx.se
| Commercial curl support up to 24x7 is available!
| Private help, bug fixes, support, ports, new features
| https://curl.se/support.html
--
Unsubscribe: https://lists.haxx.se/listinfo/curl-library
Etiquette: https://curl.se/mail/etiquette.html