[PATCH 3/4] ws2_32/tests: Add comprehensive socket option validity tests
Zebediah Figura (she/her)
zfigura at codeweavers.com
Wed Aug 11 10:17:49 CDT 2021
This is a good idea in principle, but a few things bother me about it:
(1) the tables are hard to read (could maybe be solved with indentation)?
(2) it would probably make sense to split get/set into separate tables,
and not to split tcp/udp? I dunno, looks potentially simpler.
(3) It's a bit ambiguous in many cases whether an operation is supported
or not. I.e. it'd be nice to have *successful* calls to most of these.
At least WSAENOPROTOOPT is relatively unambiguous, but I can't tell if a
given operation always returns WSAEINVAL (like, say, the hardcoded IPv4
option 41) or only returns WSAEINVAL because we're not using the right
arguments.
Which is to say that maybe a table isn't that great after all :-/
(4) It's also kind of unclear what /* win7 */ means: broken on win7, or
before win7, or up to and including win7, or what?
More information about the wine-devel
mailing list