https://gitlab.synchro.net/main/sbbs/-/commit/dbbfabf1b133b4116d69d4a13c38acfe1 607a500#note_6343

Why would we specify a socktype here (e.g. limiting to TCP sockets, in theory)?

I would think for a function that's just resolving names to addresses, it wouldn't care what the transport protocol "the socket" is, so "Specifying 0 in this field indicates that socket addresses of any type can be returned by getaddrinfo()" would be more appropriate.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/dbbfabf1b133b4116d69d4a13c38acfe1 607a500#note_6344

This looks like a bug: missing `== 0` here?
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/dbbfabf1b133b4116d69d4a13c38acfe1 607a500#note_6345

Yep, it does.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/dbbfabf1b133b4116d69d4a13c38acfe1 607a500#note_6347

But we don't actually want a separate answer for each socktype, and is seems like that would be valid.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)

https://gitlab.synchro.net/main/sbbs/-/commit/dbbfabf1b133b4116d69d4a13c38acfe1 607a500#note_6348

That said, I don't actually know why it's set.
--- SBBSecho 3.23-Linux
* Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)