Thanks Alan! I have already pushed those changes - but I'll retain this trick for the next batch. FWIW I saw that the default value for the backlog in the impl was 50, so that's what I'd been using. But not having to specify it at all is indeed a better solution.
best regards, -- daniel On 27/04/2019 15:29, Alan Bateman wrote:
One minor comment is that the 3-arg constructor requires you to specify a value for the backlog (or <= 0 to use an implementation default). An alternative that I used in other tests is to use the no-arg constructor and then bind to the loopback. It just avoids need to think about the backlog, otherwise the same.