(Modbus): дописал использование std::move для функций transport_msg().

extensions/include/MTR.h

@@ -36,48 +36,48 @@ namespace MTR
         mtT_Str8
     };
     // -------------------------------------------------------------------------
-    std::string type2str( MTRType t );            /*!< преоразование строки в тип */
-    MTRType str2type( const std::string& s );    /*!< преобразование названия в строку */
+    std::string type2str( MTRType t );          /*!< преоразование строки в тип */
+    MTRType str2type( const std::string& s );   /*!< преобразование названия в строку */
     int wsize( MTRType t );                     /*!< длина данных в словах */
     // -------------------------------------------------------------------------
     // Информация
-    const ModbusRTU::ModbusData regModelNumber    = 0x01;
-    const ModbusRTU::ModbusData regSerialNumber    = 0x09;
-    
+    const ModbusRTU::ModbusData regModelNumber  = 0x01;
+    const ModbusRTU::ModbusData regSerialNumber = 0x09;
+
     std::string getModelNumber( ModbusRTUMaster* mb, ModbusRTU::ModbusAddr addr );
     std::string getSerialNumber( ModbusRTUMaster* mb, ModbusRTU::ModbusAddr addr );
     // -------------------------------------------------------------------------
     // Настройки связи (чтение - read03, запись - write06)
     const ModbusRTU::ModbusData regUpdateConfiguration = 53;
-    const ModbusRTU::ModbusData regAddress        = 55;
-    const ModbusRTU::ModbusData regBaudRate        = 56;
-    const ModbusRTU::ModbusData regStopBit        = 57; /* 0 - Stop bit, 1 - Stop bits */
-    const ModbusRTU::ModbusData regParity        = 58;
-    const ModbusRTU::ModbusData regDataBits        = 59;
+    const ModbusRTU::ModbusData regAddress   = 55;
+    const ModbusRTU::ModbusData regBaudRate  = 56;
+    const ModbusRTU::ModbusData regStopBit   = 57; /* 0 - Stop bit, 1 - Stop bits */
+    const ModbusRTU::ModbusData regParity    = 58;
+    const ModbusRTU::ModbusData regDataBits  = 59;
 
     enum mtrBaudRate
     {
-        br1200         = 0,
-        br2400         = 1,
-        br4800         = 2,
-        br9600         = 3,
-        br19200     = 4,
-        br38400     = 5,
-        br57600        = 6,
-        br115200    = 7
+        br1200    = 0,
+        br2400    = 1,
+        br4800    = 2,
+        br9600    = 3,
+        br19200   = 4,
+        br38400   = 5,
+        br57600   = 6,
+        br115200  = 7
     };
-    
+
     enum mtrParity
     {
-        mpNoParity        = 0,
-        mpOddParity     = 1,
-        mpEvenParity     = 2
+        mpNoParity   = 0,
+        mpOddParity  = 1,
+        mpEvenParity = 2
     };
 
     enum mtrDataBits
     {
-        db8Bits    = 0,
-        db7Bits    = 1
+        db8Bits = 0,
+        db7Bits = 1
     };
 
     bool setAddress( ModbusRTUMaster* mb, ModbusRTU::ModbusAddr addr, ModbusRTU::ModbusAddr newAddr );
@@ -129,7 +129,7 @@ namespace MTR
             static MTRType type(){ return mtT1; }
             // ------------------------------------------
             unsigned short val;
-    };    
+    };
     std::ostream& operator<<(std::ostream& os, T1& t );
     // -------------------------------------------------------------------------
     class T2
@@ -162,7 +162,7 @@ namespace MTR
             // ------------------------------------------
             // конструкторы на разные случаи...
             T3(){ memset(raw.v,0,sizeof(raw.v)); }
-            
+
             T3( signed int i ){ raw.val = i; }
 
             T3( unsigned short v1, unsigned short v2 )
@@ -170,7 +170,7 @@ namespace MTR
                 raw.v[0] = v1;
                 raw.v[1] = v2;
             }
-            
+
             T3( const ModbusRTU::ModbusData* data, int size )
             {
                 if( size >= u2size )
@@ -207,7 +207,7 @@ namespace MTR
                 memcpy(c,&v1,sizeof(c));
                 sval = std::string(c);
             }
-            
+
             T4( const ModbusRTU::ModbusData* data ):
                 raw(data[0])
             {
@@ -215,7 +215,7 @@ namespace MTR
                 memcpy(c,&(data[0]),sizeof(c));
                 sval     = std::string(c);
             }
-            
+
             ~T4(){}
             // ------------------------------------------
             /*! размер в словах */
@@ -257,7 +257,7 @@ namespace MTR
             {
                 raw.lval = v;
             }
-            
+
             T5( const ModbusRTU::ModbusData* data, int size )
             {
                 if( size >= u2size )
@@ -310,7 +310,7 @@ namespace MTR
             {
                 raw.lval = v;
             }
-            
+
             T6( const ModbusRTU::ModbusData* data, int size )
             {
                 if( size >= u2size )
@@ -363,7 +363,7 @@ namespace MTR
             {
                 raw.lval = v;
             }
-            
+
             T7( const ModbusRTU::ModbusData* data, int size )
             {
                 if( size >= u2size )
@@ -421,7 +421,7 @@ namespace MTR
                     raw.v[0] = data[1];
                 }
             }
-            
+
             inline unsigned short day(){ return raw.u2.day; }
             inline unsigned short mon(){ return raw.u2.mon; }
             inline unsigned short hour(){ return raw.u2.hour; }
@@ -522,7 +522,7 @@ namespace MTR
                     raw.v[1] = data[0];
                 }
             }
-            
+
             inline unsigned short year(){ return raw.u2.year; }
             inline unsigned short mon(){ return raw.u2.mon; }
             inline unsigned short day(){ return raw.u2.day; }
@@ -538,7 +538,7 @@ namespace MTR
     };
     std::ostream& operator<<(std::ostream& os, T10& t );
     // --------------------------------------------------------------------------
-    
+
     class T16
     {
         public:
@@ -650,7 +650,7 @@ namespace MTR
             // ------------------------------------------
             operator float(){ return raw.val; }
             operator long(){ return lroundf(raw.val); }
-            
+
             F1mem raw;
     };
     std::ostream& operator<<(std::ostream& os, F1& t );

include/modbus/ModbusTypes.h

@@ -175,16 +175,16 @@ namespace ModbusRTU
     {
         ModbusByte ecode;
         ModbusCRC crc;
-        
+
         // ------- from slave -------
         ErrorRetMessage( ModbusMessage& m );
         ErrorRetMessage& operator=( ModbusMessage& m );
         void init( ModbusMessage& m );
-        
+
         // ------- to master -------
         ErrorRetMessage( ModbusAddr _from, ModbusByte _func, ModbusByte ecode );
 
-        /*! преобразование для посылки в сеть */    
+        /*! преобразование для посылки в сеть */
         ModbusMessage transport_msg();
 
         /*! размер данных(после заголовка) у данного типа сообщения 

src/Communications/Modbus/ModbusTypes.cc

@@ -260,7 +260,7 @@ ModbusMessage ErrorRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; // szData();
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 std::ostream& ModbusRTU::operator<<(std::ostream& os, ErrorRetMessage& m )
@@ -309,7 +309,7 @@ ModbusMessage ReadCoilMessage::transport_msg()
      memcpy(&(mm.data[last]),&crc,szCRC);
 
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 ReadCoilMessage::ReadCoilMessage( ModbusMessage& m )
@@ -434,7 +434,7 @@ ModbusData DataBits16::mdata()
             udata |= 1<<i;
     }
 
-    return udata;
+    return std::move(udata);
 }
 // -------------------------------------------------------------------------
 const DataBits16& DataBits16::operator=( const ModbusData& r )
@@ -568,7 +568,7 @@ ModbusMessage ReadCoilRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t ReadCoilRetMessage::szData()
@@ -620,7 +620,7 @@ ModbusMessage ReadInputStatusMessage::transport_msg()
      memcpy(&(mm.data[last]),&crc,szCRC);
 
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 ReadInputStatusMessage::ReadInputStatusMessage( ModbusMessage& m )
@@ -768,7 +768,7 @@ ModbusMessage ReadInputStatusRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t ReadInputStatusRetMessage::szData()
@@ -823,7 +823,7 @@ ModbusMessage ReadOutputMessage::transport_msg()
     // длина сообщения...
     mm.len = szData();
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 ReadOutputMessage::ReadOutputMessage( ModbusMessage& m )
@@ -950,10 +950,10 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind=0;
     bcnt    = count*sizeof(ModbusData);
-        
+
     // copy bcnt
     memcpy(&mm.data,&bcnt,sizeof(bcnt));
     ind+=sizeof(bcnt);
@@ -979,7 +979,7 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
     ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader+sizeof(bcnt)+bcnt );
 
 //    crc = SWAPSHORT(crc);
-    
+
     // копируем CRC (последний элемент). Без переворачивания...
      memcpy(&(mm.data[ind]),&crc,szCRC);
     ind+=szCRC;
@@ -987,7 +987,7 @@ ModbusMessage ReadOutputRetMessage::transport_msg()
     // длина сообщения...
     mm.len = ind; 
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t ReadOutputRetMessage::szData()
@@ -1007,8 +1007,6 @@ std::ostream& ModbusRTU::operator<<(std::ostream& os, ReadOutputRetMessage* m )
 }
 // -------------------------------------------------------------------------
 // -------------------------------------------------------------------------
-
-// -------------------------------------------------------------------------
 ReadInputMessage::ReadInputMessage( ModbusAddr a, ModbusData s, ModbusData c ):
     start(s),
     count(c)
@@ -1042,7 +1040,7 @@ ModbusMessage ReadInputMessage::transport_msg()
 
     // длина сообщения...
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 ReadInputMessage::ReadInputMessage( ModbusMessage& m )
@@ -1062,7 +1060,7 @@ void ReadInputMessage::init( ModbusMessage& m )
     assert( m.func == fnReadInputRegisters );
     memset(this,0,sizeof(*this));
     memcpy(this,&m,sizeof(*this)); // m.len
-    
+
     // переворачиваем слова
     start = SWAPSHORT(start);
     count = SWAPSHORT(count);
@@ -1166,10 +1164,10 @@ ModbusMessage ReadInputRetMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind=0;
     bcnt    = count*sizeof(ModbusData);
-        
+
     // copy bcnt
     memcpy(&mm.data,&bcnt,sizeof(bcnt));
     ind+=sizeof(bcnt);
@@ -1195,7 +1193,7 @@ ModbusMessage ReadInputRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t ReadInputRetMessage::szData()
@@ -1289,7 +1287,7 @@ ModbusMessage ForceCoilsMessage::transport_msg()
 
     // копируем заголовок
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind = 0;
 
     // данные (переворачиваем байты)
@@ -1317,7 +1315,7 @@ ModbusMessage ForceCoilsMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 
@@ -1477,7 +1475,7 @@ ModbusMessage ForceCoilsRetMessage::transport_msg()
     // длина сообщения...
     mm.len = szData();
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 std::ostream& ModbusRTU::operator<<(std::ostream& os, ForceCoilsRetMessage& m )
@@ -1524,7 +1522,7 @@ ModbusMessage WriteOutputMessage::transport_msg()
 
     // копируем заголовок
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind = 0;
 
     // данные (переворачиваем байты)
@@ -1559,7 +1557,7 @@ ModbusMessage WriteOutputMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 
@@ -1734,7 +1732,7 @@ ModbusMessage WriteOutputRetMessage::transport_msg()
     // длина сообщения...
     mm.len = szData();
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 std::ostream& ModbusRTU::operator<<(std::ostream& os, WriteOutputRetMessage& m )
@@ -1766,9 +1764,9 @@ ModbusMessage ForceSingleCoilMessage::transport_msg()
     // пересчитываем CRC по перевёрнутым данным
     ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader+last );
     // копируем CRC (последний элемент). Без переворачивания...
-     memcpy(&(mm.data[last]),&crc,szCRC);
+    memcpy(&(mm.data[last]),&crc,szCRC);
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // --------------------------------------------------------------------------------
 
@@ -1904,7 +1902,7 @@ ModbusMessage ForceSingleCoilRetMessage::transport_msg()
     // длина сообщения...
     mm.len = szData();
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 std::ostream& ModbusRTU::operator<<(std::ostream& os, ForceSingleCoilRetMessage& m )
@@ -1937,9 +1935,9 @@ ModbusMessage WriteSingleOutputMessage::transport_msg()
     // пересчитываем CRC по перевёрнутым данным
     ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader+last );
     // копируем CRC (последний элемент). Без переворачивания...
-     memcpy(&(mm.data[last]),&crc,szCRC);
+    memcpy(&(mm.data[last]),&crc,szCRC);
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // --------------------------------------------------------------------------------
 
@@ -2077,12 +2075,12 @@ ModbusMessage WriteSingleOutputRetMessage::transport_msg()
     ModbusData crc = checkCRC( (ModbusByte*)(&mm), szModbusHeader+sizeof(d) );
 
     // копируем CRC (последний элемент). Без переворачивания...
-     memcpy(&(mm.data[last]),&crc,szCRC);
+    memcpy(&(mm.data[last]),&crc,szCRC);
 
     // длина сообщения...
     mm.len = szData();
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 std::ostream& ModbusRTU::operator<<(std::ostream& os, WriteSingleOutputRetMessage& m )
@@ -2236,7 +2234,7 @@ ModbusMessage DiagnosticMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind=0;
     // copy bcnt
     ModbusData d = SWAPSHORT(subf);
@@ -2264,7 +2262,7 @@ ModbusMessage DiagnosticMessage::transport_msg()
 
     // длина сообщения...
     mm.len = szData(); 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t DiagnosticMessage::szData()
@@ -2360,7 +2358,7 @@ ModbusMessage MEIMessageRDI::transport_msg()
 
     // длина сообщения...
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 
@@ -2566,7 +2564,7 @@ ModbusMessage MEIMessageRetRDI::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     mm.data[0] = type;
     mm.data[1] = devID;
     mm.data[2] = conformity;
@@ -2593,7 +2591,7 @@ ModbusMessage MEIMessageRetRDI::transport_msg()
 
     // длина сообщения...
     mm.len = ind;
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t MEIMessageRetRDI::szData()
@@ -2727,10 +2725,10 @@ ModbusMessage JournalCommandRetMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind = 0;
     bcnt    = count*sizeof(ModbusData);
-        
+
     // copy bcnt
     memcpy(&mm.data,&bcnt,sizeof(bcnt));
     ind += sizeof(bcnt);
@@ -2759,7 +2757,7 @@ ModbusMessage JournalCommandRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 size_t JournalCommandRetMessage::szData()
@@ -2982,7 +2980,7 @@ ModbusMessage SetDateTimeMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-/*    
+/*
     mm.data[0] = hour;
     mm.data[1] = min;
     mm.data[2] = sec;
@@ -3000,7 +2998,7 @@ ModbusMessage SetDateTimeMessage::transport_msg()
      memcpy(&(mm.data[bcnt]),&crc,szCRC);
     // длина сообщения...
     mm.len = szData(); // bcnt + szCRC
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 SetDateTimeRetMessage::SetDateTimeRetMessage( ModbusMessage& m )
@@ -3056,7 +3054,7 @@ ModbusMessage SetDateTimeRetMessage::transport_msg()
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
 
-/*    
+/*
     mm.data[0] = hour;
     mm.data[1] = min;
     mm.data[2] = sec;
@@ -3077,7 +3075,7 @@ ModbusMessage SetDateTimeRetMessage::transport_msg()
     // длина сообщения...
     mm.len = szData(); // bcnt + szCRC
 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 RemoteServiceMessage::RemoteServiceMessage( ModbusMessage& m )
@@ -3176,7 +3174,7 @@ ModbusMessage RemoteServiceRetMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind = 0;
     bcnt    = count*sizeof(ModbusByte);
 
@@ -3199,7 +3197,7 @@ ModbusMessage RemoteServiceRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind; 
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 ReadFileRecordMessage::ReadFileRecordMessage( ModbusMessage& m )
@@ -3305,7 +3303,7 @@ ModbusMessage FileTransferMessage::transport_msg()
 
     // длина сообщения...
     mm.len = szData();
-    return mm;
+    return std::move(mm);
 }
 // -------------------------------------------------------------------------
 FileTransferMessage::FileTransferMessage( ModbusMessage& m )
@@ -3433,7 +3431,7 @@ ModbusMessage FileTransferRetMessage::transport_msg()
 
     // копируем заголовок и данные
     memcpy(&mm,this,szModbusHeader);
-    
+
     int ind = 0;
     bcnt = szData() - szCRC - 1; // -1 - это сам байт содержащий количество байт (bcnt)...
 
@@ -3465,7 +3463,7 @@ ModbusMessage FileTransferRetMessage::transport_msg()
 
     // длина сообщения...
     mm.len = ind;
-    return mm;
+    return std::move(mm);
 }
 // -----------------------------------------------------------------------
 std::ostream& ModbusRTU::operator<<(std::ostream& os, FileTransferRetMessage& m )