Merge pull request #37 from hintjens/master

Fixed test_many_sockets

NEWS

@@ -1,3 +1,13 @@
+0MQ version 4.0.3 stable, released on 2013/11/24
+================================================
+
+Bug Fixes
+---------
+
+* Fixed test_many_sockets case, which failed when process socket limit
+  was 1024.
+
+
 0MQ version 4.0.2 stable, released on 2013/11/24
 ================================================
 

include/zmq.h

@@ -31,7 +31,7 @@
 /*  Version macros for compile-time API version detection                     */
 #define ZMQ_VERSION_MAJOR 4
 #define ZMQ_VERSION_MINOR 0
-#define ZMQ_VERSION_PATCH 2
+#define ZMQ_VERSION_PATCH 3
 
 #define ZMQ_MAKE_VERSION(major, minor, patch) \
     ((major) * 10000 + (minor) * 100 + (patch))

tests/test_many_sockets.cpp

@@ -23,34 +23,27 @@
 #include <stdlib.h>
 #include <vector>
 
-
 void test_system_max ()
 {
     // Keep allocating sockets until we run out of system resources
-
     const int no_of_sockets = 2 * 65536;
     void *ctx = zmq_ctx_new ();
     zmq_ctx_set (ctx, ZMQ_MAX_SOCKETS, no_of_sockets);
     std::vector <void*> sockets;
 
-    while (true)
-    {
+    while (true) {
         void *socket = zmq_socket (ctx, ZMQ_PAIR);
         if (!socket)
             break;
-
         sockets.push_back (socket);
     }
-
     assert ((int) sockets.size () < no_of_sockets);
 
-    // System is out of resources, further calls to zmq_socket should return NULL.
-    for (unsigned int i = 0; i < 10; ++i)
-    {
+    //  System is out of resources, further calls to zmq_socket should return NULL
+    for (unsigned int i = 0; i < 10; ++i) {
         void *socket = zmq_socket (ctx, ZMQ_PAIR);
         assert (socket == NULL);
     }
-
     // Clean up.
     for (unsigned int i = 0; i < sockets.size (); ++i)
         zmq_close (sockets [i]);
@@ -60,30 +53,26 @@ void test_system_max ()
 
 void test_zmq_default_max ()
 {
-    // Keep allocating sockets until we hit the default zeromq limit
-
+    //  Keep allocating sockets until we hit the default limit
     void *ctx = zmq_ctx_new ();
     std::vector<void*> sockets;
 
-    while (true)
-    {
+    while (true) {
         void *socket = zmq_socket (ctx, ZMQ_PAIR);
         if (!socket)
             break;
-
         sockets.push_back (socket);
     }
+    //  We may stop sooner if system has fewer available sockets
+    assert (sockets.size () <= ZMQ_MAX_SOCKETS_DFLT);
 
-    assert(sockets.size() == ZMQ_MAX_SOCKETS_DFLT);
-
-    // At zeromq max, further calls to zmq_socket should return NULL.
-    for (unsigned int i = 0; i < 10; ++i)
-    {
+    //  Further calls to zmq_socket should return NULL
+    for (unsigned int i = 0; i < 10; ++i) {
         void *socket = zmq_socket (ctx, ZMQ_PAIR);
         assert (socket == NULL);
     }
 
-    // Clean up.
+    //  Clean up
     for (unsigned int i = 0; i < sockets.size (); ++i)
         zmq_close (sockets [i]);
 

tests/testutil.hpp

@@ -22,7 +22,7 @@
 
 #include "../include/zmq.h"
 #include "../include/zmq_utils.h"
-#include "platform.hpp"
+#include "../src/platform.hpp"
 
 //  This defines the settle time used in tests; raise this if we
 //  get test failures on slower systems due to binds/connects not