Fix warnings

* [-Wunused-parameter] * [-Wunused-but-set-variable]
FairRootGroup · Jun 10, 2021 · b6ff7d8 · b6ff7d8
1 parent 23270cb
commit b6ff7d8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/MiscCommon/Process.h b/MiscCommon/Process.h
@@ -412,7 +412,11 @@ namespace MiscCommon
      */
     typedef std::vector<pid_t> vectorPid_t;
 
+#if defined(__APPLE__)
     inline vectorPid_t getprocbyname(const std::string& _Srv, bool _filterForRealUserID = false)
+#else
+    inline vectorPid_t getprocbyname(const std::string& _Srv, bool /*_filterForRealUserID = false*/)
+#endif
     {
         vectorPid_t retVal;
 #if defined(__APPLE__)

diff --git a/dds-protocol-lib/src/BaseSMChannelImpl.h b/dds-protocol-lib/src/BaseSMChannelImpl.h
@@ -35,7 +35,6 @@
                                                                                           \
         using namespace dds;                                                              \
         using namespace dds::protocol_api;                                                \
-        bool processed = true;                                                            \
         ECmdType currentCmd = static_cast<ECmdType>(_currentMsg->header().m_cmd);         \
         SSenderInfo sender;                                                               \
         sender.m_ID = _currentMsg->header().m_ID;                                         \
@@ -50,7 +49,7 @@
     {                                                                                                              \
         typedef typename SCommandAttachmentImpl<msg>::ptr_t attahcmentPtr_t;                                       \
         attahcmentPtr_t attachmentPtr = SCommandAttachmentImpl<msg>::decode(_currentMsg);                          \
-        processed = func(attachmentPtr, sender);                                                                   \
+        bool processed = func(attachmentPtr, sender);                                                              \
         if (!processed)                                                                                            \
         {                                                                                                          \
             if (!handlerExists(msg))                                                                               \
@@ -69,7 +68,6 @@
 #define SM_MESSAGE_HANDLER_DISPATCH(msg)                                                                         \
     case msg:                                                                                                    \
     {                                                                                                            \
-        processed = false;                                                                                       \
         typedef typename SCommandAttachmentImpl<msg>::ptr_t attahcmentPtr_t;                                     \
         attahcmentPtr_t attachmentPtr = SCommandAttachmentImpl<msg>::decode(_currentMsg);                        \
         LOG(MiscCommon::debug) << "Dispatching " << g_cmdToString[msg];                                          \
@@ -107,12 +105,11 @@
                                                                                                                        \
         using namespace dds;                                                                                           \
         using namespace dds::protocol_api;                                                                             \
-        bool processed = true;                                                                                         \
         SSenderInfo sender;                                                                                            \
         sender.m_ID = _currentMsg->header().m_ID;                                                                      \
         try                                                                                                            \
         {                                                                                                              \
-            processed = func(_currentMsg, sender);                                                                     \
+            bool processed = func(_currentMsg, sender);                                                                \
             if (!processed)                                                                                            \
             {                                                                                                          \
                 if (!handlerExists(ECmdType::cmdRAW_MSG))                                                              \