Fix further effc++ and reorder warnings.

fairmq/FairMQDevice.cxx

@@ -47,7 +47,6 @@ static void CallSignalHandler(int signal)
 
 FairMQDevice::FairMQDevice()
     : fChannels()
-    , fConfig(nullptr)
     , fId()
     , fNetworkInterface()
     , fMaxInitializationAttempts(120)
@@ -57,6 +56,7 @@ FairMQDevice::FairMQDevice()
     , fLogIntervalInMs(1000)
     , fCmdSocket(nullptr)
     , fTransportFactory(nullptr)
+    , fConfig(nullptr)
     , fInitialValidationFinished(false)
     , fInitialValidationCondition()
     , fInitialValidationMutex()

fairmq/nanomsg/FairMQSocketNN.cxx

@@ -165,7 +165,7 @@ int64_t FairMQSocketNN::Send(const vector<unique_ptr<FairMQMessage>>& msgVec, co
     msgpack::packer<msgpack::sbuffer> packer(&sbuf);
 
     // pack all parts into a single msgpack simple buffer
-    for (int i = 0; i < msgVec.size(); ++i)
+    for (unsigned int i = 0; i < msgVec.size(); ++i)
     {
         static_cast<FairMQMessageNN*>(msgVec[i].get())->fReceiving = false;
         packer.pack_bin(msgVec[i]->GetSize());
@@ -270,7 +270,7 @@ int64_t FairMQSocketNN::Receive(vector<unique_ptr<FairMQMessage>>& msgVec, const
 
         // offset to be used by msgpack to handle separate chunks
         size_t offset = 0;
-        while (offset != nbytes) // continue until all parts have been read
+        while (offset != static_cast<size_t>(nbytes)) // continue until all parts have been read
         {
             // vector of chars to hold blob (unlike char*/void* this type can be converted to by msgpack)
             std::vector<char> buf;

fairmq/tools/FairMQDDSTools.h

@@ -21,6 +21,11 @@ using namespace dds::intercom_api;
 // container to hold channel config and corresponding dds key values
 struct DDSConfig
 {
+    DDSConfig()
+        : subChannels()
+        , ddsValues()
+    {}
+
     // container of sub channels, e.g. 'i' in data[i]
     vector<FairMQChannel*> subChannels;
     // dds values for the channel
@@ -111,7 +116,7 @@ void HandleConfigViaDDS(TMQDevice& device)
                     if (mi->second.subChannels.size() == mi->second.ddsValues.size())
                     {
                         auto it = mi->second.ddsValues.begin();
-                        for (int i = 0; i < mi->second.subChannels.size(); ++i)
+                        for (unsigned int i = 0; i < mi->second.subChannels.size(); ++i)
                         {
                           mi->second.subChannels.at(i)->UpdateAddress(it->second);
                           ++it;