Attention is currently required from: neels, fixeria, pespin.
6 comments:
File src/socket.c:
Patch Set #9, Line 206: buf_len
There is no 'buf_len' anymore. […]
Done
Patch Set #9, Line 211: const char **hosts, size_t host_count,
cosmetic: alignment needs to be fixed here
Done
I think it makes sense to assert() here that either **hosts or **addrs is passed, not both at the sa […]
Seems rather pointless to me: that's a static function with only few invocations, not part of public API so we don't have to insure against user who failed to read the docs.
Patch Set #9, Line 236: if (hosts[i])
Before your patch this function would print "0.0.0.0" if hosts[i] is NULL. […]
That seems like odd side-effect to me for which I couldn't find any tests. Having NULL input we can't guess whether IPv4 or IPv6 address to show. Moreover, print helper seems like a wrong place for such guesswork in a first place - the caller have more context information and can pass proper default explicitly. I'll update commit message to make it more visible.
Patch Set #9, Line 241: LOGL_DEBUG
LOGL_ERROR? Not sure if we need to log anything here though.
Done
Better do sizeof(strbuf).
Done
To view, visit change 28993. To unsubscribe, or for help writing mail filters, visit settings.