If you comment out the changes in NodeShared.cc, then the bench test will hang forever because messages are lost.
This is a problem without a good solution for everyone. If the buffer is too small we can loose messages but if the buffer is too big (or infinite), we can leak forever and run out of memory. I’m OK with this solution because I understand it solves some issue but I think we should expose this functionality to the user as an option in the same way ZeroMQ does. We don’t have to do it in this PR, just a suggestion. The example works for me with the patch.