From bde18f3ede841346321b4dd0e87109d4880672c6 Mon Sep 17 00:00:00 2001 From: Luca Boccassi Date: Sat, 18 May 2019 17:02:06 +0100 Subject: [PATCH] Problem: comments before ifdefs are wrongly formatted Solution: now that clang-format 6+ is used in the CI, fix it --- src/options.cpp | 8 ++++---- src/socket_base.cpp | 8 ++++---- src/socks_connecter.cpp | 2 +- src/tcp_connecter.cpp | 6 +++--- src/udp_engine.cpp | 4 ++-- src/ypipe.hpp | 6 +++--- src/ypipe_conflate.hpp | 6 +++--- src/zmq.cpp | 2 +- 8 files changed, 21 insertions(+), 21 deletions(-) diff --git a/src/options.cpp b/src/options.cpp index d5f18018..b6ae3ea2 100644 --- a/src/options.cpp +++ b/src/options.cpp @@ -761,10 +761,10 @@ int zmq::options_t::setsockopt (int option_, break; } - // TODO mechanism should either be set explicitly, or determined when - // connecting. currently, it depends on the order of setsockopt calls - // if there is some inconsistency, which is confusing. in addition, - // the assumed or set mechanism should be queryable (as a socket option) + // TODO mechanism should either be set explicitly, or determined when + // connecting. currently, it depends on the order of setsockopt calls + // if there is some inconsistency, which is confusing. in addition, + // the assumed or set mechanism should be queryable (as a socket option) #if defined(ZMQ_ACT_MILITANT) // There is no valid use case for passing an error back to the application diff --git a/src/socket_base.cpp b/src/socket_base.cpp index f1934fd2..e1b2fae7 100644 --- a/src/socket_base.cpp +++ b/src/socket_base.cpp @@ -353,9 +353,9 @@ int zmq::socket_base_t::check_protocol (const std::string &protocol_) const return -1; } - // Check whether socket type and transport protocol match. - // Specifically, multicast protocols can't be combined with - // bi-directional messaging patterns (socket types). + // Check whether socket type and transport protocol match. + // Specifically, multicast protocols can't be combined with + // bi-directional messaging patterns (socket types). #if defined ZMQ_HAVE_OPENPGM || defined ZMQ_HAVE_NORM if ((protocol_ == "pgm" || protocol_ == "epgm" || protocol_ == "norm") && options.type != ZMQ_PUB && options.type != ZMQ_SUB @@ -896,7 +896,7 @@ int zmq::socket_base_t::connect (const char *endpoint_uri_) } } - // TBD - Should we check address for ZMQ_HAVE_NORM??? + // TBD - Should we check address for ZMQ_HAVE_NORM??? #ifdef ZMQ_HAVE_OPENPGM if (protocol == "pgm" || protocol == "epgm") { diff --git a/src/socks_connecter.cpp b/src/socks_connecter.cpp index 9e068e8e..71c28a0e 100644 --- a/src/socks_connecter.cpp +++ b/src/socks_connecter.cpp @@ -255,7 +255,7 @@ int zmq::socks_connecter_t::connect_to_proxy () } } - // Connect to the remote peer. + // Connect to the remote peer. #if defined ZMQ_HAVE_VXWORKS rc = ::connect (_s, (sockaddr *) tcp_addr->addr (), tcp_addr->addrlen ()); #else diff --git a/src/tcp_connecter.cpp b/src/tcp_connecter.cpp index 863f223f..67cca590 100644 --- a/src/tcp_connecter.cpp +++ b/src/tcp_connecter.cpp @@ -219,7 +219,7 @@ int zmq::tcp_connecter_t::open () return -1; } - // Connect to the remote peer. + // Connect to the remote peer. #if defined ZMQ_HAVE_VXWORKS rc = ::connect (_s, (sockaddr *) tcp_addr->addr (), tcp_addr->addrlen ()); #else @@ -230,8 +230,8 @@ int zmq::tcp_connecter_t::open () return 0; } - // Translate error codes indicating asynchronous connect has been - // launched to a uniform EINPROGRESS. + // Translate error codes indicating asynchronous connect has been + // launched to a uniform EINPROGRESS. #ifdef ZMQ_HAVE_WINDOWS const int last_error = WSAGetLastError (); if (last_error == WSAEINPROGRESS || last_error == WSAEWOULDBLOCK) diff --git a/src/udp_engine.cpp b/src/udp_engine.cpp index 4707773b..6276fe26 100644 --- a/src/udp_engine.cpp +++ b/src/udp_engine.cpp @@ -222,8 +222,8 @@ void zmq::udp_engine_t::plug (io_thread_t *io_thread_, session_base_t *session_) bool multicast = udp_addr->is_mcast (); if (multicast) { - // Multicast addresses should be allowed to bind to more than - // one port as all ports should receive the message + // Multicast addresses should be allowed to bind to more than + // one port as all ports should receive the message #ifdef SO_REUSEPORT rc = setsockopt (_fd, SOL_SOCKET, SO_REUSEPORT, reinterpret_cast (&on), sizeof (on)); diff --git a/src/ypipe.hpp b/src/ypipe.hpp index 0f2b9e72..34dc5b69 100644 --- a/src/ypipe.hpp +++ b/src/ypipe.hpp @@ -62,9 +62,9 @@ template class ypipe_t : public ypipe_base_t // just to keep ICC and code checking tools from complaining. inline virtual ~ypipe_t () {} - // Following function (write) deliberately copies uninitialised data - // when used with zmq_msg. Initialising the VSM body for - // non-VSM messages won't be good for performance. + // Following function (write) deliberately copies uninitialised data + // when used with zmq_msg. Initialising the VSM body for + // non-VSM messages won't be good for performance. #ifdef ZMQ_HAVE_OPENVMS #pragma message save diff --git a/src/ypipe_conflate.hpp b/src/ypipe_conflate.hpp index 9d75eb9a..ac8141b9 100644 --- a/src/ypipe_conflate.hpp +++ b/src/ypipe_conflate.hpp @@ -53,9 +53,9 @@ template class ypipe_conflate_t : public ypipe_base_t // just to keep ICC and code checking tools from complaining. inline virtual ~ypipe_conflate_t () {} - // Following function (write) deliberately copies uninitialised data - // when used with zmq_msg. Initialising the VSM body for - // non-VSM messages won't be good for performance. + // Following function (write) deliberately copies uninitialised data + // when used with zmq_msg. Initialising the VSM body for + // non-VSM messages won't be good for performance. #ifdef ZMQ_HAVE_OPENVMS #pragma message save diff --git a/src/zmq.cpp b/src/zmq.cpp index 5bdef399..f75e30a5 100644 --- a/src/zmq.cpp +++ b/src/zmq.cpp @@ -712,7 +712,7 @@ const char *zmq_msg_gets (const zmq_msg_t *msg_, const char *property_) return NULL; } - // Polling. +// Polling. #if defined ZMQ_HAVE_POLLER inline int zmq_poller_poll (zmq_pollitem_t *items_, int nitems_, long timeout_)