Problem: redundant parentheses in return statements

Solution: remove them

builds/cmake/Modules/ZMQSourceRunChecks.cmake

@@ -42,7 +42,7 @@ macro(zmq_check_o_cloexec)
 int main(int argc, char *argv [])
 {
     int s = open (\"/dev/null\", O_CLOEXEC | O_RDONLY);
-    return (s == -1);
+    return s == -1;
 }
 "
     ZMQ_HAVE_O_CLOEXEC)

src/norm_engine.hpp

@@ -87,7 +87,7 @@ class norm_engine_t : public io_object_t, public i_engine
         // These are used to feed data to decoder
         // and its underlying "msg" buffer
         char *AccessBuffer () { return (char *) (buffer_ptr + buffer_count); }
-        size_t GetBytesNeeded () const { return (buffer_size - buffer_count); }
+        size_t GetBytesNeeded () const { return buffer_size - buffer_count; }
         void IncrementBufferCount (size_t count) { buffer_count += count; }
         msg_t *AccessMsg () { return zmq_decoder->msg (); }
         // This invokes the decoder "decode" method
@@ -106,7 +106,7 @@ class norm_engine_t : public io_object_t, public i_engine
             void Append (NormRxStreamState &item);
             void Remove (NormRxStreamState &item);
 
-            bool IsEmpty () const { return (NULL == head); }
+            bool IsEmpty () const { return NULL == head; }
 
             void Destroy ();
 

src/pipe.cpp

@@ -546,7 +546,7 @@ bool zmq::pipe_t::check_hwm () const
 {
     const bool full =
       _hwm > 0 && _msgs_written - _peers_msgs_read >= uint64_t (_hwm);
-    return (!full);
+    return !full;
 }
 
 void zmq::pipe_t::send_hwms_to_peer (int inhwm_, int outhwm_)

src/select.cpp

@@ -542,7 +542,7 @@ void zmq::select_t::cleanup_retired ()
 
 bool zmq::select_t::is_retired_fd (const fd_entry_t &entry_)
 {
-    return (entry_.fd == retired_fd);
+    return entry_.fd == retired_fd;
 }
 
 zmq::select_t::family_entry_t::family_entry_t () : has_retired (false)