Style fixes

src/decoder.hpp

@@ -143,7 +143,8 @@ namespace zmq
             }
         }
 
-        inline bool message_ready_size (size_t msg_sz){
+        inline bool message_ready_size (size_t msg_sz)
+        {
             zmq_assert (false);
             return false;
         }

src/raw_encoder.cpp

@@ -58,7 +58,6 @@ bool zmq::raw_encoder_t::raw_message_size_ready ()
 
 bool zmq::raw_encoder_t::raw_message_ready ()
 {
-
     //  Destroy content of the old message.
     int rc = in_progress.close ();
     errno_assert (rc == 0);

src/raw_encoder.hpp

@@ -40,7 +40,6 @@
 namespace zmq
 {
 
-
     //  Encoder for 0MQ framing protocol. Converts messages into data batches.
 
     class raw_encoder_t : public encoder_base_t <raw_encoder_t>

src/router.cpp

@@ -78,8 +78,8 @@ void zmq::router_t::xattach_pipe (pipe_t *pipe_, bool icanhasall_)
 int zmq::router_t::xsetsockopt (int option_, const void *optval_,
     size_t optvallen_)
 {
-    if (option_ != ZMQ_ROUTER_MANDATORY &&
-        option_ != ZMQ_ROUTER_RAW_SOCK) {
+    if (option_ != ZMQ_ROUTER_MANDATORY
+    &&  option_ != ZMQ_ROUTER_RAW_SOCK) {
         errno = EINVAL;
         return -1;
     }
@@ -93,10 +93,9 @@ int zmq::router_t::xsetsockopt (int option_, const void *optval_,
             options.recv_identity = false;
             options.raw_sock = true;
         }
-
-    }else{
-        mandatory = *static_cast <const int*> (optval_);
     }
+    else
+        mandatory = *static_cast <const int*> (optval_);
     return 0;
 }
 
@@ -186,7 +185,7 @@ int zmq::router_t::xsend (msg_t *msg_, int flags_)
         return 0;
     }
 
-    // ignore the MORE flag for raw-sock or assert?
+    //  Ignore the MORE flag for raw-sock or assert?
     if (options.raw_sock)
         msg_->reset_flags (msg_t::more);
 
@@ -349,12 +348,13 @@ bool zmq::router_t::identify_peer (pipe_t *pipe_)
     blob_t identity;
     bool ok;
 
-    if(options.raw_sock){ // always assign identity for raw-socket
+    if (options.raw_sock) { //  Always assign identity for raw-socket
         unsigned char buf [5];
         buf [0] = 0;
         put_uint32 (buf + 1, next_peer_id++);
         identity = blob_t (buf, sizeof buf);
-    }else{
+    }
+    else {
         msg.init ();
         ok = pipe_->read (&msg);
         if (!ok)

src/session_base.cpp

@@ -120,10 +120,10 @@ zmq::session_base_t::session_base_t (class io_thread_t *io_thread_,
     identity_received (false),
     addr (addr_)
 {
-    // identities are not exchanged for raw sockets
+    //  Identities are not exchanged for raw sockets
     if (options.raw_sock) {
-        identity_sent  = (true);
-        identity_received = (true);
+        identity_sent = true;
+        identity_received = true;
     }
 }
 

src/stream_engine.cpp

@@ -146,7 +146,8 @@ void zmq::stream_engine_t::plug (io_thread_t *io_thread_,
 
         // disable handshaking for raw socket
         handshaking = false;
-    }else{
+    }
+    else {
         //  Send the 'length' and 'flags' fields of the identity message.
         //  The 'length' field is encoded in the long format.
         outpos = greeting_output_buffer;
@@ -216,12 +217,12 @@ void zmq::stream_engine_t::in_event ()
     }
 
     if (options.raw_sock) {
-        if(insize == 0 || !decoder->message_ready_size(insize)){
+        if (insize == 0 || !decoder->message_ready_size (insize))
             processed = 0;
-        }else{
+        else
             processed = decoder->process_buffer (inpos, insize);
     }
-    }else{
+    else {
         //  Push the data to the decoder.
         processed = decoder->process_buffer (inpos, insize);
     }

tests/test_raw_sock.cpp

@@ -36,73 +36,67 @@
 //ToDo: Windows?
 const char *test_str = "TEST-STRING";
 
-
-int tcp_client(){
-
-    int sockfd, portno;
+int tcp_client ()
+{
     struct sockaddr_in serv_addr;
     struct hostent *server;
 
-    portno = 5555;
+    const int portno = 5555;
 
-    sockfd = socket(AF_INET, SOCK_STREAM, 0);
+    int sockfd = socket (AF_INET, SOCK_STREAM, 0);
     assert (sockfd >= 0);
     server = gethostbyname ("localhost");
     assert (server);
 
-    bzero((char *) &serv_addr, sizeof(serv_addr));
+    bzero (&serv_addr, sizeof serv_addr);
     serv_addr.sin_family = AF_INET;
-    bcopy((char *)server->h_addr,
-         (char *)&serv_addr.sin_addr.s_addr,
-         server->h_length);
+    bcopy (server->h_addr, &serv_addr.sin_addr.s_addr, server->h_length);
     serv_addr.sin_port = htons (portno);
 
-    if (connect(sockfd,(struct sockaddr *) &serv_addr,sizeof(serv_addr)) < 0)
-    	assert(0);
-    int nodelay = 1;
-    int rc = setsockopt (sockfd, IPPROTO_TCP, TCP_NODELAY, (char*) &nodelay,
-        sizeof (int));
+    int rc = connect (sockfd, (struct sockaddr *) &serv_addr, sizeof serv_addr);
+    assert (rc == 0);
+    int nodelay = 1;
+    rc = setsockopt (sockfd, IPPROTO_TCP, TCP_NODELAY, (char*) &nodelay,
+        sizeof nodelay);
     assert (rc == 0);
-
 
     return sockfd;
 }
 
-void tcp_client_write(int sockfd, const void *buf, int buf_len){
+void tcp_client_write (int sockfd, const void *buf, int buf_len)
+{
     assert (buf);
     int n = write (sockfd, buf, buf_len);
     assert (n >= 0);
 }
 
-void tcp_client_read(int sockfd){
+void tcp_client_read (int sockfd)
+{
     struct timeval tm;
     tm.tv_sec = 1;
     tm.tv_usec = 0;
     fd_set r;
 
-    int  sr;
     char buffer [16];
 
     FD_ZERO (&r);
     FD_SET (sockfd, &r);
 
-    if ((sr = select(sockfd + 1, &r, NULL, NULL, &tm)) <= 0)
-    {
-    	assert(0);
-    }
+    int sr = select (sockfd + 1, &r, NULL, NULL, &tm);
+    assert (sr > 0);
 
     int n = read (sockfd, buffer, 16);
     assert (n > 0);
     assert (memcmp (buffer, test_str, strlen (test_str)) == 0);
 }
 
-
-void tcp_client_close(int sockfd){
+void tcp_client_close (int sockfd)
+{
     close (sockfd);
 }
 
-
-int main(){
+int main ()
+{
     fprintf (stderr, "test_raw_sock running...\n");
 
     zmq_msg_t message;
@@ -112,10 +106,11 @@ int main(){
     void *ctx = zmq_init (1);
     assert (ctx);
 
-    int raw_sock = 1, rc = 0;
     void *sb = zmq_socket (ctx, ZMQ_ROUTER);
     assert (sb);
-    rc = zmq_setsockopt( sb, ZMQ_ROUTER_RAW_SOCK, &raw_sock, sizeof(int));
+
+    int raw_sock = 1;
+    int rc = zmq_setsockopt (sb, ZMQ_ROUTER_RAW_SOCK, &raw_sock, sizeof raw_sock);
     assert (rc == 0);
     rc = zmq_bind (sb, "tcp://127.0.0.1:5555");
     assert (rc == 0);
@@ -134,15 +129,12 @@ int main(){
 
     tcp_client_write (sock_fd, test_str, strlen (test_str));
     zmq_poll (items, 1, 500);
-    if (items [0].revents & ZMQ_POLLIN) {
+    assert (items [0].revents & ZMQ_POLLIN);
     int n = zmq_msg_recv (&id, sb, 0);
     assert (n > 0);
     n = zmq_msg_recv (&message, sb, 0);
     assert (n > 0);
     assert (memcmp (zmq_msg_data (&message), test_str, strlen (test_str)) == 0);
-    }else{
-        assert(0);
-    }
 
     zmq_msg_send (&id, sb, ZMQ_SNDMORE);
     zmq_msg_send (&message, sb, ZMQ_SNDMORE); // SNDMORE option is ignored
@@ -153,7 +145,6 @@ int main(){
     zmq_msg_close (&id);
     zmq_msg_close (&message);
 
-
     zmq_close (sb);
     zmq_term (ctx);
 
@@ -161,7 +152,3 @@ int main(){
 
     return 0;
 }
-
-
-
-