Problem: unused variables warnings

Solution: fix them

Author: nehalkpatel

src/ip.cpp

@@ -210,6 +210,9 @@ void zmq::set_socket_priority (fd_t s_, int priority_)
       setsockopt (s_, SOL_SOCKET, SO_PRIORITY,
                   reinterpret_cast<char *> (&priority_), sizeof (priority_));
     errno_assert (rc == 0);
+#else
+    LIBZMQ_UNUSED (s_);
+    LIBZMQ_UNUSED (priority_);
 #endif
 }
 

src/zmq.cpp

@@ -1739,6 +1739,8 @@ int zmq_proxy_steerable (void *frontend_,
                          void *capture_,
                          void *control_)
 {
+    LIBZMQ_UNUSED (capture_);
+    LIBZMQ_UNUSED (control_);
     if (!frontend_ || !backend_) {
         errno = EFAULT;
         return -1;

src/zmtp_engine.cpp

@@ -371,6 +371,9 @@ bool zmq::zmtp_engine_t::handshake_v3_x (const bool downgrade_sub_)
         error (protocol_error);
         return false;
     }
+#ifndef ZMQ_HAVE_CURVE
+    LIBZMQ_UNUSED (downgrade_sub_);
+#endif
     _next_msg = &zmtp_engine_t::next_handshake_command;
     _process_msg = &zmtp_engine_t::process_handshake_command;
 

tests/test_bind_fuzzer.cpp

@@ -59,6 +59,9 @@ void test_bind_fuzzer ()
 
 int main (int argc, char **argv)
 {
+    LIBZMQ_UNUSED (argc);
+    LIBZMQ_UNUSED (argv);
+
     setup_test_environment ();
 
     UNITY_BEGIN ();

tests/test_bind_null_fuzzer.cpp

@@ -76,6 +76,8 @@ void test_bind_null_fuzzer ()
 
 int main (int argc, char **argv)
 {
+    LIBZMQ_UNUSED (argc);
+    LIBZMQ_UNUSED (argv);
     setup_test_environment ();
 
     UNITY_BEGIN ();

tests/test_connect_fuzzer.cpp

@@ -46,6 +46,9 @@ void test_connect_fuzzer ()
 
 int main (int argc, char **argv)
 {
+    LIBZMQ_UNUSED (argc);
+    LIBZMQ_UNUSED (argv);
+
     setup_test_environment ();
 
     UNITY_BEGIN ();

tests/test_connect_null_fuzzer.cpp

@@ -81,6 +81,9 @@ void test_connect_null_fuzzer ()
 
 int main (int argc, char **argv)
 {
+    LIBZMQ_UNUSED (argc);
+    LIBZMQ_UNUSED (argv);
+
     setup_test_environment ();
 
     UNITY_BEGIN ();

tests/test_poller.cpp

@@ -297,6 +297,7 @@ void call_poller_wait_all_null_event_fails_event_count_nonzero (void *poller_)
 
 void call_poller_wait_all_null_event_fails_event_count_zero (void *poller_)
 {
+    LIBZMQ_UNUSED (poller_);
 #if 0
     //  TODO this causes an assertion, which is not consistent if the number
     //  of events may be 0, the pointer should be allowed to by NULL in that

tests/test_reconnect_options.cpp

@@ -49,6 +49,7 @@ void reconnect_default ()
     int rc = get_monitor_event_with_timeout (sub_mon, &event, &event_address,
                                              2 * 1000);
     assert (rc == -1);
+    LIBZMQ_UNUSED (rc);
 
     //  Close sub
     //  TODO why does this use zero_linger?
@@ -102,6 +103,7 @@ void reconnect_success ()
     int rc = get_monitor_event_with_timeout (sub_mon, &event, &event_address,
                                              SETTLE_TIME);
     assert (rc == -1);
+    LIBZMQ_UNUSED (rc);
 
     //  Now re-bind pub socket and wait for re-connect
     pub = test_context_socket (ZMQ_PUB);

tests/test_socks.cpp

@@ -77,6 +77,7 @@ int remote_connect (int socket, uint32_t addr, uint16_t port)
 void *setup_socks_server (char *socks_server_address,
                           int socks_server_address_len)
 {
+    LIBZMQ_UNUSED (socks_server_address_len);
     fprintf (stderr, "socks_server: setup socks server\n");
     int server_fd =
       bind_socket_resolve_port ("127.0.0.1", "0", socks_server_address);