Изменения после проверки cppcheck (performance)

Utilities/Admin/admin.cc

@@ -462,7 +462,7 @@ int setValue( const string& args, UInterface &ui, Configuration* conf )
     typedef std::list<UniSetTypes::ParamSInfo> SList;
     SList sl = UniSetTypes::getSInfoList(args, conf);
     cout << "====== setValue ======" << endl;
-    for( SList::iterator it=sl.begin(); it!=sl.end(); it++ )
+    for( SList::iterator it=sl.begin(); it!=sl.end(); ++it )
     {
         try
         {
@@ -508,7 +508,7 @@ int getValue( const string& args, UInterface &ui, Configuration* conf )
     typedef std::list<UniSetTypes::ParamSInfo> SList;
     SList sl = UniSetTypes::getSInfoList( args, UniSetTypes::conf );
     cout << "====== getValue ======" << endl;
-    for( SList::iterator it=sl.begin(); it!=sl.end(); it++ )
+    for( SList::iterator it=sl.begin(); it!=sl.end(); ++it )
     {    
         try
         {
@@ -551,7 +551,7 @@ int getCalibrate( const std::string& args, UInterface &ui )
       typedef std::list<UniSetTypes::ParamSInfo> SList;
     SList sl = UniSetTypes::getSInfoList( args, UniSetTypes::conf );
     cout << "====== getCalibrate ======" << endl;
-    for( SList::iterator it=sl.begin(); it!=sl.end(); it++ )
+    for( SList::iterator it=sl.begin(); it!=sl.end(); ++it )
     {
         if( it->si.node == DefaultObjectId )
             it->si.node = conf->getLocalNode();
@@ -581,7 +581,7 @@ int getRawValue( const std::string& args, UInterface &ui )
     typedef std::list<UniSetTypes::ParamSInfo> SList;
     SList sl = UniSetTypes::getSInfoList( args, UniSetTypes::conf );
     cout << "====== getRawValue ======" << endl;
-    for( SList::iterator it=sl.begin(); it!=sl.end(); it++ )
+    for( SList::iterator it=sl.begin(); it!=sl.end(); ++it )
     {    
         if( it->si.node == DefaultObjectId )
             it->si.node = conf->getLocalNode();

extensions/RRDServer/RRDServer.cc

@@ -74,7 +74,7 @@ void RRDServer::initRRD( xmlNode* cnode, int tmID )
         throw SystemError(err.str());
     }
 
-    for(;it_rra.getCurrent(); it_rra++ )
+    for(;it_rra.getCurrent(); ++it_rra )
     {
         string rra( it_rra.getProp("rra") );
         if( rra.empty() )

extensions/SMDBServer/SMDBServer.cc

@@ -10,7 +10,7 @@ using namespace UniSetTypes;
 using namespace UniSetExtensions;
 // -----------------------------------------------------------------------------
 SMDBServer::SMDBServer( UniSetTypes::ObjectId objId, UniSetTypes::ObjectId shmId, SharedMemory* ic,
-            const string prefix ):
+            const string& prefix ):
 DBServer_MySQL(objId),
 prefix(prefix)
 {
@@ -178,7 +178,7 @@ void SMDBServer::help_print( int argc, const char* const* argv )
 // -----------------------------------------------------------------------------
 SMDBServer* SMDBServer::init_smdbserver( int argc, const char* const* argv, 
                                             UniSetTypes::ObjectId icID, SharedMemory* ic, 
-                                            const std::string prefix )
+                                            const std::string& prefix )
 {
     string name = conf->getArgParam("--" + prefix + "-name","DBServer");
     if( name.empty() )

extensions/SMDBServer/SMDBServer.h

@@ -12,13 +12,13 @@ class SMDBServer:
 {
     public:
         SMDBServer( UniSetTypes::ObjectId objId, UniSetTypes::ObjectId shmID, SharedMemory* ic=0,
-                    const std::string prefix="dbserver" );
+                    const std::string& prefix="dbserver" );
         virtual ~SMDBServer();
 
         /*! глобальная функция для инициализации объекта */
         static SMDBServer* init_smdbserver( int argc, const char* const* argv, 
                             UniSetTypes::ObjectId shmID, SharedMemory* ic=0,
-                            const std::string prefix="dbserver" );
+                            const std::string& prefix="dbserver" );
 
         /*! глобальная функция для вывода help-а */
         static void help_print( int argc, const char* const* argv );

include/modbus/ModbusTypes.h

@@ -207,7 +207,7 @@ namespace ModbusRTU
         operator ModbusByte();
         ModbusByte mbyte();
 
-        bool operator[]( const int i ){ return b[i]; }
+        bool operator[]( const size_t i ){ return b[i]; }
 
         std::bitset<BitsPerByte> b;
     };
@@ -218,7 +218,7 @@ namespace ModbusRTU
     struct DataBits16
     {
         DataBits16( ModbusData d );
-        DataBits16( std::string s ); // example "1000111110001111"
+        DataBits16( const std::string& s ); // example "1000111110001111"
         DataBits16();
 
         const DataBits16& operator=(const ModbusData& r);
@@ -226,7 +226,7 @@ namespace ModbusRTU
         operator ModbusData();
         ModbusData mdata();
 
-        bool operator[]( const int i ){ return b[i]; }
+        bool operator[]( const size_t i ){ return b[i]; }
         void set( int n, bool s ){ b.set(n,s); }
 
         std::bitset<BitsPerData> b;
@@ -322,7 +322,7 @@ namespace ModbusRTU
         }
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! преобразование для посылки в сеть */
         ModbusMessage transport_msg();
@@ -416,7 +416,7 @@ namespace ModbusRTU
         }
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! преобразование для посылки в сеть */    
         ModbusMessage transport_msg();
@@ -495,7 +495,7 @@ namespace ModbusRTU
         }
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! преобразование для посылки в сеть */
         ModbusMessage transport_msg();
@@ -583,7 +583,7 @@ namespace ModbusRTU
         void swapData();
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! преобразование для посылки в сеть */
         ModbusMessage transport_msg();
@@ -651,7 +651,7 @@ namespace ModbusRTU
         void init( ModbusMessage& m );
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! размер предварительного заголовка 
          * (после основного до фактических данных) 
@@ -741,7 +741,7 @@ namespace ModbusRTU
         void init( ModbusMessage& m );
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! размер предварительного заголовка 
          * (после основного до фактических данных) 
@@ -828,7 +828,7 @@ namespace ModbusRTU
         void init( ModbusMessage& m );
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! размер предварительного заголовка 
          * (после основного до фактических данных) 
@@ -911,7 +911,7 @@ namespace ModbusRTU
         void init( ModbusMessage& m );
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! размер предварительного заголовка 
          * (после основного до фактических данных) 
@@ -1015,7 +1015,7 @@ namespace ModbusRTU
         }
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
         
         /*! преобразование для посылки в сеть */    
         ModbusMessage transport_msg();
@@ -1143,7 +1143,7 @@ namespace ModbusRTU
         }
         
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
         
         /*! преобразование для посылки в сеть */
         ModbusMessage transport_msg();
@@ -1211,7 +1211,7 @@ namespace ModbusRTU
         }
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
         
         /*! преобразование для посылки в сеть */    
         ModbusMessage transport_msg();
@@ -1314,7 +1314,7 @@ namespace ModbusRTU
         void init( ModbusMessage& m );
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! размер предварительного заголовка 
          * (после основного до фактических данных) 
@@ -1355,7 +1355,7 @@ namespace ModbusRTU
             { return ( count >= sizeof(data) ); }
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
         
         /*! преобразование для посылки в сеть */    
         ModbusMessage transport_msg();
@@ -1390,7 +1390,7 @@ namespace ModbusRTU
         void init( ModbusMessage& m );
 
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
 
         /*! размер предварительного заголовка 
          * (после основного до фактических данных) 
@@ -1473,7 +1473,7 @@ namespace ModbusRTU
         void clear();
         
         /*! размер данных(после заголовка) у данного типа сообщения */
-        int szData();
+        size_t szData();
         
         /*! преобразование для посылки в сеть */    
         ModbusMessage transport_msg();

src/Communications/Modbus/ModbusServer.cc

@@ -1462,15 +1462,16 @@ ModbusRTU::mbErrCode ModbusServer::replySetDateTime( ModbusRTU::SetDateTimeMessa
 
     if( gettimeofday(&set,&tz) == 0 )
     {
-        struct tm*  t = localtime(&set.tv_sec);
-        t->tm_sec     = query.sec;
-        t->tm_min     = query.min;
-        t->tm_hour     = query.hour;
-        t->tm_mday     = query.day;
-        t->tm_mon     = query.mon-1;
-//        t->tm_year     = (query.century>19) ? query.year + query.century*10 - 1900 : query.year;
-        t->tm_year     = (query.century>19) ? query.year + 2000 - 1900 : query.year;
-        set.tv_sec = mktime(t);
+        struct tm  t;
+        localtime_r(&set.tv_sec,&t);
+        t.tm_sec     = query.sec;
+        t.tm_min     = query.min;
+        t.tm_hour     = query.hour;
+        t.tm_mday     = query.day;
+        t.tm_mon     = query.mon-1;
+//        t.tm_year     = (query.century>19) ? query.year + query.century*10 - 1900 : query.year;
+        t.tm_year     = (query.century>19) ? query.year + 2000 - 1900 : query.year;
+        set.tv_sec = mktime(&t);
         set.tv_usec = 0;
 
         if( settimeofday(&set,&tz)==0 )

src/Communications/Modbus/ModbusTypes.cc

@@ -392,7 +392,7 @@ const DataBits& DataBits::operator=( const ModbusByte& r )
 std::ostream& ModbusRTU::operator<<(std::ostream& os, DataBits& d )
 {
     os << "[";
-    for( unsigned int i = (int)d.b.size()-1; i>=0; i-- )
+    for( int i=d.b.size()-1; i>=0; i-- )
         os << d.b[i];
     os << "]";
 
@@ -404,7 +404,7 @@ std::ostream& ModbusRTU::operator<<(std::ostream& os, DataBits* d )
     return os << (*d);
 }
 // -------------------------------------------------------------------------
-DataBits16::DataBits16( std::string s ):
+DataBits16::DataBits16( const std::string& s ):
 b(s)
 {
 
@@ -439,7 +439,8 @@ ModbusData DataBits16::mdata()
 // -------------------------------------------------------------------------
 const DataBits16& DataBits16::operator=( const ModbusData& r )
 {
-    for( unsigned int i=0; i<b.size(); i++ )
+    const size_t sz = b.size();
+    for( unsigned int i=0; i<sz; i++ )
         b[i] = r&(1<<i);
 
     return (*this);
@@ -448,7 +449,7 @@ const DataBits16& DataBits16::operator=( const ModbusData& r )
 std::ostream& ModbusRTU::operator<<(std::ostream& os, DataBits16& d )
 {
     os << "[";
-    for( unsigned int i=(int)d.b.size()-1; i>=0; i-- )
+    for( int i=d.b.size()-1; i>=0; i-- )
         os << d.b[i];
     os << "]";
 
@@ -570,7 +571,7 @@ ModbusMessage ReadCoilRetMessage::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int ReadCoilRetMessage::szData()
+size_t ReadCoilRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+bcnt+szCRC;
@@ -770,7 +771,7 @@ ModbusMessage ReadInputStatusRetMessage::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int ReadInputStatusRetMessage::szData()
+size_t ReadInputStatusRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+bcnt+szCRC;
@@ -989,7 +990,7 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int ReadOutputRetMessage::szData()
+size_t ReadOutputRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+count*sizeof(ModbusData)+szCRC;
@@ -1197,7 +1198,7 @@ ModbusMessage ReadInputRetMessage::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int ReadInputRetMessage::szData()
+size_t ReadInputRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+count*sizeof(ModbusData)+szCRC;
@@ -1370,7 +1371,7 @@ bool ForceCoilsMessage::checkFormat()
     return ( func==fnForceMultipleCoils );
 }
 // -------------------------------------------------------------------------
-int ForceCoilsMessage::szData()
+size_t ForceCoilsMessage::szData()
 {
     return szHead()+bcnt+szCRC;
 }
@@ -1617,7 +1618,7 @@ bool WriteOutputMessage::checkFormat()
     return ( (bcnt==(quant*sizeof(ModbusData))) && (func==fnWriteOutputRegisters) );
 }
 // -------------------------------------------------------------------------
-int WriteOutputMessage::szData()
+size_t WriteOutputMessage::szData()
 {
     return szHead()+bcnt+szCRC;
 }
@@ -1812,7 +1813,7 @@ bool ForceSingleCoilMessage::checkFormat()
     return (func==fnForceSingleCoil);
 }
 // -------------------------------------------------------------------------
-int ForceSingleCoilMessage::szData()
+size_t ForceSingleCoilMessage::szData()
 {
     return szHead()+sizeof(ModbusData)+szCRC;
 }
@@ -1984,7 +1985,7 @@ bool WriteSingleOutputMessage::checkFormat()
     return ( (func==fnWriteOutputSingleRegister) );
 }
 // -------------------------------------------------------------------------
-int WriteSingleOutputMessage::szData()
+size_t WriteSingleOutputMessage::szData()
 {
     return szHead()+sizeof(ModbusData)+szCRC;
 }
@@ -2266,7 +2267,7 @@ ModbusMessage DiagnosticMessage::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int DiagnosticMessage::szData()
+size_t DiagnosticMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(subf)+count*sizeof(ModbusData)+szCRC;
@@ -2595,7 +2596,7 @@ ModbusMessage MEIMessageRetRDI::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int MEIMessageRetRDI::szData()
+size_t MEIMessageRetRDI::szData()
 {
     // заголовочные поля + фактическое число данных + контрольная сумма
     return 6 + bcnt + szCRC;
@@ -2761,7 +2762,7 @@ ModbusMessage JournalCommandRetMessage::transport_msg()
     return mm;
 }
 // -------------------------------------------------------------------------
-int JournalCommandRetMessage::szData()
+size_t JournalCommandRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+count*sizeof(ModbusData)+szCRC;
@@ -3102,7 +3103,7 @@ void RemoteServiceMessage::init( ModbusMessage& m )
     memcpy(&crc,&(m.data[m.len-szCRC]),szCRC);
 }
 // -------------------------------------------------------------------------
-int RemoteServiceMessage::szData()
+size_t RemoteServiceMessage::szData()
 {
     return szHead()+bcnt+szCRC;
 }
@@ -3162,7 +3163,7 @@ void RemoteServiceRetMessage::clear()
     bcnt    = 0;
 }
 // -------------------------------------------------------------------------
-int RemoteServiceRetMessage::szData()
+size_t RemoteServiceRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+count*sizeof(ModbusByte)+szCRC;
@@ -3249,7 +3250,7 @@ void ReadFileRecordMessage::init( ModbusMessage& m )
     }
 }
 // -------------------------------------------------------------------------
-int ReadFileRecordMessage::szData()
+size_t ReadFileRecordMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(bcnt)+count*sizeof(SubRequest)+szCRC;
@@ -3419,7 +3420,7 @@ int FileTransferRetMessage::getDataLen( ModbusMessage& m )
     return m.data[0];
 }
 // -----------------------------------------------------------------------
-int FileTransferRetMessage::szData()
+size_t FileTransferRetMessage::szData()
 {
     // фактическое число данных + контрольная сумма
     return sizeof(ModbusByte)*2+sizeof(ModbusData)*3+dlen+szCRC;

src/ObjectRepository/UniSetTypes.cc

@@ -237,7 +237,7 @@ using namespace UniSetTypes;
         std::list<UniSetTypes::ParamSInfo> res;
 
         std::list<std::string> l = UniSetTypes::explode_str(str,',');
-        for( std::list<std::string>::iterator it=l.begin(); it!=l.end(); it++ )
+        for( std::list<std::string>::iterator it=l.begin(); it!=l.end(); ++it )
         {
             UniSetTypes::ParamSInfo item;