minor fixes, make style

extensions/ModbusMaster/MBExchange.cc

@@ -2674,7 +2674,7 @@ bool MBExchange::initDeviceInfo( RTUDeviceMap& m, ModbusRTU::ModbusAddr a, UniXM
 	dinfo << myname << "(initDeviceInfo): add addr=" << ModbusRTU::addr2str(a) << endl;
 	int tout = it.getPIntProp("timeout", ptTimeout.getInterval() );
 
-	d->second->resp_Delay.set(0,tout); // ставим время на отпускание.. см. checkRespond()
+	d->second->resp_Delay.set(0, tout); // ставим время на отпускание.. см. checkRespond()
 	d->second->resp_invert = it.getIntProp("invert");
 	return true;
 }
@@ -3005,6 +3005,7 @@ bool MBExchange::poll()
 					  << " for sensors: ";
 				print_plist(dlog()->level3(), it->second->slst)
 						<< endl << " err: " << ex << endl;
+
 				if( ex.err == ModbusRTU::erTimeOut && !d->ask_every_reg )
 					break;
 			}
@@ -3070,16 +3071,16 @@ bool MBExchange::RTUDevice::checkRespond()
 {
 	bool prev = resp_state;
 
-	resp_state = resp_Delay.check( prev_numreply!=numreply );
+	resp_state = resp_Delay.check( prev_numreply != numreply );
 
 	prev_numreply.store(numreply);
 
-	return (prev!=resp_state);
+	return (prev != resp_state);
 }
 // -----------------------------------------------------------------------------
 void MBExchange::updateRespondSensors()
 {
-	for( const auto& it1: rmap )
+	for( const auto& it1 : rmap )
 	{
 		RTUDevice* d(it1.second);
 

extensions/ModbusMaster/MBTCPMaster.cc

@@ -13,9 +13,7 @@ using namespace UniSetExtensions;
 MBTCPMaster::MBTCPMaster( UniSetTypes::ObjectId objId, UniSetTypes::ObjectId shmId,
 						  const std::shared_ptr<SharedMemory> ic, const std::string& prefix ):
 	MBExchange(objId, shmId, ic, prefix),
-	force_disconnect(true),
-	mbtcp(nullptr),
-	pollThread(0)
+	force_disconnect(true)
 {
 	if( objId == DefaultObjectId )
 		throw UniSetTypes::SystemError("(MBTCPMaster): objId=-1?!! Use --" + prefix + "-name" );
@@ -55,7 +53,7 @@ MBTCPMaster::MBTCPMaster( UniSetTypes::ObjectId objId, UniSetTypes::ObjectId shm
 	else
 		ic->addReadItem( sigc::mem_fun(this, &MBTCPMaster::readItem) );
 
-	pollThread = new ThreadCreator<MBTCPMaster>(this, &MBTCPMaster::poll_thread);
+	pollThread = make_shared<ThreadCreator<MBTCPMaster>>(this, &MBTCPMaster::poll_thread);
 	pollThread->setFinalAction(this, &MBTCPMaster::final_thread);
 
 	if( dlog()->is_info() )
@@ -71,9 +69,6 @@ MBTCPMaster::~MBTCPMaster()
 		if( pollThread->isRunning() )
 			pollThread->join();
 	}
-
-	delete pollThread;
-	mbtcp.reset();
 }
 // -----------------------------------------------------------------------------
 std::shared_ptr<ModbusClient> MBTCPMaster::initMB( bool reopen )
@@ -195,9 +190,6 @@ void MBTCPMaster::sigterm( int signo )
 
 		if( pollThread->isRunning() )
 			pollThread->join();
-
-		delete pollThread;
-		pollThread = 0;
 	}
 
 	try

extensions/ModbusMaster/MBTCPMaster.h

@@ -225,7 +225,7 @@ class MBTCPMaster:
 
 		// т.к. TCP может "зависнуть" на подключении к недоступному узлу
 		// делаем опрос в отдельном потоке
-		ThreadCreator<MBTCPMaster>* pollThread; /*!< поток опроса */
+		std::shared_ptr<ThreadCreator<MBTCPMaster>> pollThread; /*!< поток опроса */
 		UniSetTypes::uniset_rwmutex tcpMutex;
 };
 // -----------------------------------------------------------------------------

extensions/ModbusMaster/MBTCPMultiMaster.cc

@@ -13,9 +13,7 @@ using namespace UniSetExtensions;
 MBTCPMultiMaster::MBTCPMultiMaster( UniSetTypes::ObjectId objId, UniSetTypes::ObjectId shmId,
 									const std::shared_ptr<SharedMemory> ic, const std::string& prefix ):
 	MBExchange(objId, shmId, ic, prefix),
-	force_disconnect(true),
-	pollThread(0),
-	checkThread(0)
+	force_disconnect(true)
 {
 	tcpMutex.setName(myname + "_tcpMutex");
 
@@ -131,9 +129,9 @@ MBTCPMultiMaster::MBTCPMultiMaster( UniSetTypes::ObjectId objId, UniSetTypes::Ob
 	else
 		ic->addReadItem( sigc::mem_fun(this, &MBTCPMultiMaster::readItem) );
 
-	pollThread = new ThreadCreator<MBTCPMultiMaster>(this, &MBTCPMultiMaster::poll_thread);
+	pollThread = make_shared<ThreadCreator<MBTCPMultiMaster>>(this, &MBTCPMultiMaster::poll_thread);
 	pollThread->setFinalAction(this, &MBTCPMultiMaster::final_thread);
-	checkThread = new ThreadCreator<MBTCPMultiMaster>(this, &MBTCPMultiMaster::check_thread);
+	checkThread = make_shared<ThreadCreator<MBTCPMultiMaster>>(this, &MBTCPMultiMaster::check_thread);
 	checkThread->setFinalAction(this, &MBTCPMultiMaster::final_thread);
 
 
@@ -153,8 +151,6 @@ MBTCPMultiMaster::~MBTCPMultiMaster()
 
 		if( pollThread->isRunning() )
 			pollThread->join();
-
-		delete pollThread;
 	}
 
 	if( checkThread )
@@ -163,15 +159,9 @@ MBTCPMultiMaster::~MBTCPMultiMaster()
 
 		if( checkThread->isRunning() )
 			checkThread->join();
-
-		delete checkThread;
 	}
 
-	for( auto& it : mblist )
-	{
-		it.mbtcp.reset(); // = nullptr;
-		mbi = mblist.rend();
-	}
+	mbi = mblist.rend();
 }
 // -----------------------------------------------------------------------------
 std::shared_ptr<ModbusClient> MBTCPMultiMaster::initMB( bool reopen )

extensions/ModbusMaster/MBTCPMultiMaster.h

@@ -286,10 +286,10 @@ class MBTCPMultiMaster:
 
 		// т.к. TCP может "зависнуть" на подключении к недоступному узлу
 		// делаем опрос в отдельном потоке
-		ThreadCreator<MBTCPMultiMaster>* pollThread; /*!< поток опроса */
+		std::shared_ptr< ThreadCreator<MBTCPMultiMaster> > pollThread; /*!< поток опроса */
 		UniSetTypes::uniset_rwmutex tcpMutex;
 
-		ThreadCreator<MBTCPMultiMaster>* checkThread; /*!< поток проверки связи по другим каналам */
+		std::shared_ptr< ThreadCreator<MBTCPMultiMaster> > checkThread; /*!< поток проверки связи по другим каналам */
 };
 // -----------------------------------------------------------------------------
 #endif // _MBTCPMultiMaster_H_

extensions/ModbusMaster/RTUExchange.cc

@@ -223,6 +223,7 @@ bool RTUExchange::poll()
 		}
 
 		int prev_numreply = d->numreply;
+
 		if( d->dtype == MBExchange::dtRTU188 )
 		{
 			if( !d->rtu )

include/modbus/ModbusTypes.h

@@ -77,11 +77,11 @@ namespace ModbusRTU
 
 	typedef unsigned long RegID;
 
-	/*! Получение уникального ID (hash?) на основе номера функции и регистра 
+	/*! Получение уникального ID (hash?) на основе номера функции и регистра
 	 *  Требования к данной функции:
-     *  1. ID > диапазона возможных регистров (>65535)
-     *  2. одинаковые регистры, но разные функции должны давать разный ID
-     *  3. регистры идущие подряд, должны давать ID идущие тоже подряд
+	 *  1. ID > диапазона возможных регистров (>65535)
+	 *  2. одинаковые регистры, но разные функции должны давать разный ID
+	 *  3. регистры идущие подряд, должны давать ID идущие тоже подряд
 	*/
 	RegID genRegID( const ModbusRTU::ModbusData r, const int fn );
 

tests/test_modbustypes.cc

@@ -118,14 +118,15 @@ TEST_CASE("genRegID", "[modbus][genRegID]" )
 	int max_reg = numeric_limits<ModbusRTU::ModbusData>::max();
 	int max_fn = numeric_limits<ModbusRTU::ModbusByte>::max();
 
-	ModbusRTU::RegID prevID = ModbusRTU::genRegID(0,0);
-	for( int f=1; f<max_fn; f++ )
+	ModbusRTU::RegID prevID = ModbusRTU::genRegID(0, 0);
+
+	for( int f = 1; f < max_fn; f++ )
 	{
-		ModbusRTU::RegID minID = ModbusRTU::genRegID(0,f);
+		ModbusRTU::RegID minID = ModbusRTU::genRegID(0, f);
 		REQUIRE( minID > prevID );
 
-		for( int r=1; r<max_reg; r++ )
-			REQUIRE( ModbusRTU::genRegID(r,f) == minID+r );
+		for( int r = 1; r < max_reg; r++ )
+			REQUIRE( ModbusRTU::genRegID(r, f) == minID + r );
 	}
 }
 #endif

tests/test_mutex.cc

@@ -198,17 +198,19 @@ TEST_CASE("uniset_rwmutex_{wr|r} thread lock", "[mutex][threadlock][basic]" )
 	g_mutex.rlock();
 
 	std::vector< std::future<bool> > vw(3);
-	for( int w=0; w<3; w++ )
-		vw.push_back( std::async(std::launch::async, writer_thread,w) );
+
+	for( int w = 0; w < 3; w++ )
+		vw.push_back( std::async(std::launch::async, writer_thread, w) );
 
 	std::vector< std::future<bool> > vr(3);
-	for( int r=0; r<5; r++ )
-		vr.push_back( std::async(std::launch::async, reader_thread,r) );
+
+	for( int r = 0; r < 5; r++ )
+		vr.push_back( std::async(std::launch::async, reader_thread, r) );
 
 	msleep(10);
 
 	// read захватывают сразу без задержек..
-	for( auto&& i: vr )
+	for( auto && i : vr )
 	{
 		if( i.valid()  )
 			REQUIRE( i.get() == true );
@@ -218,7 +220,7 @@ TEST_CASE("uniset_rwmutex_{wr|r} thread lock", "[mutex][threadlock][basic]" )
 	g_mutex.unlock();
 
 	// проверяем что write-ы завершают работу.
-	for( auto& i: vw )
+	for( auto& i : vw )
 	{
 		if( i.valid() )
 			REQUIRE( i.get() == true );