MINOR: stream-int: make it clear that si_ops cannot be null
authorWilly Tarreau <w@1wt.eu>
Wed, 7 Nov 2018 10:28:12 +0000 (11:28 +0100)
committerWilly Tarreau <w@1wt.eu>
Sun, 11 Nov 2018 09:18:37 +0000 (10:18 +0100)
commit57f08bb63b6bd79a1d3e46880c65cffa67a9a260
tree6831132932663ed96ee90e9c7cdc8c2eb917e3e1
parent43e69dc1eb10f67a414b2db7bc2c8153b3fb0f6e
MINOR: stream-int: make it clear that si_ops cannot be null

There was an ambiguity in which functions of the si_ops struct could be
null or not. only ->update doesn't exist in one of the si_ops (the
embedded one), all others are always defined. ->shutr and ->shutw were
never tested. However ->chk_rcv() and ->chk_snd() were tested, causing
confusion about the proper way to wake the other side up if undefined
(which never happens).

Let's update the comments to state these functions are mandatory and
remove the offending checks.
include/proto/stream_interface.h
include/types/stream_interface.h