zeromq / azmq

C++ language binding library integrating ZeroMQ with Boost Asio
Boost Software License 1.0
319 stars 109 forks source link

azmq/detail/context_ops.hpp: Assign ZMQ_MAX_SOCKETS to max_sockets #191

Closed jvanns closed 2 years ago

jvanns commented 2 years ago

Without this patch, max_sockets references ZMQ_MAXMSGSIZE which isn't quite right ;) Probably a typo - lets just quickly fix it!

I've tested this locally by subscribing to >1000 topics to avoid the proverbial "Too many open files" and already ensuring that my 'ulimit -n <>' soft & hard limits allow for the extra sockets.

aboseley commented 2 years ago

Nice catch!