Мелкие правки (чтобы cppcheck не ругался) и др.

extensions/ModbusSlave/tests/mbslave-tests.cc

@@ -300,7 +300,7 @@ TEST_CASE("(0x06): write register outputs or memories","[modbus][mbslave][mbtcps
 #endif
 
 #if 0
-    \TODO  Переписать реализацию MBSlave... ввести понятие nbit.
+/*! \TODO  Переписать реализацию MBSlave... ввести понятие nbit. */
 TEST_CASE("(0x0F): force multiple coils","[modbus][mbslave][mbtcpslave]")
 {
     WARN("FIXME: 'force coil status'. Use 'nbit'?"):

include/CallbackTimer.h

@@ -84,7 +84,7 @@ class CallbackTimer
     public:
 
         /*! Максимальное количество таймеров */
-        static const int MAXCallbackTimer = 20;
+        static const unsigned int MAXCallbackTimer = 20;
 
         /*! прототип функции вызова */
         typedef void(Caller::* Action)( int id );

src/ObjectRepository/UniSetManager.cc

@@ -120,8 +120,10 @@ UniSetManager::~UniSetManager()
     catch(...){}
 
 #if 0
+/*
 # НЕ МЫ СОЗДАВАЛИ.. НЕ НАМ И УНИЧТОЖАТЬ!
 # нужно перейти на shared_ptr<>..
+*/
     for( auto& i: olist )
     {
         try

tests/test_delaytimer.cc

@@ -53,6 +53,18 @@ TEST_CASE("DelayTimer", "[DelayTimer]" )
         msleep(50); // в сумме уже 20+50=70 > 60, значит должно "отпустить"
         CHECK_FALSE( dt.check(false) );
         CHECK_FALSE( dt.get() );
+
+        dt.reset();
+        CHECK_FALSE( dt.check(true) );
+        msleep(50);
+        CHECK_FALSE( dt.check(true) );
+        dt.reset();
+        CHECK_FALSE( dt.check(true) );
+        msleep(60);
+        CHECK_FALSE( dt.check(true) );
+        msleep(60);
+        CHECK( dt.check(true) );
+        CHECK( dt.get() );
     }
 
     SECTION( "Debounce" )
@@ -106,4 +118,15 @@ TEST_CASE("DelayTimer", "[DelayTimer]" )
         CHECK( dt1.get() );
         CHECK_FALSE( dt2.get() );
     }
+
+    SECTION( "Other" )
+    {
+        DelayTimer dt(100,50);
+        REQUIRE( dt.getOnDelay() == 100 );
+        REQUIRE( dt.getOffDelay() == 50 );
+
+        dt.set(150,200);
+        REQUIRE( dt.getOnDelay() == 150 );
+        REQUIRE( dt.getOffDelay() == 200 );
+    }
 }

tests/test_modbustypes.cc

@@ -71,7 +71,7 @@ TEST_CASE("Modbus  helpers", "[modbus][helpers]" )
 }
 
 #if 0
- Надо ещё подумать как тут протестировать
+/*! \TODO Надо ещё подумать как тут протестировать */
 TEST_CASE("dat2f", "[modbus]" )
 {
     using namespace ModbusRTU;

testsuite/testsuite-coverage-common.mk

@@ -8,7 +8,7 @@ cov: all
 	make check
 	$(LCOV) -c -d $(abs_srcdir) -o $(abs_srcdir)/.coverage.run
 	$(LCOV) -d $(abs_srcdir) -a $(abs_srcdir)/.coverage.base -a $(abs_srcdir)/.coverage.run -o $(abs_srcdir)/.coverage.total
-	$(LCOV) -r $(abs_srcdir)/.coverage.total '/usr/include/c++/*' '/usr/include/cc++/*' '/usr/include/omniORB4/*' '/usr/include/sigc++-2.0/sigc++/*'\
+	$(LCOV) -r $(abs_srcdir)/.coverage.total '/usr/include/catch.hpp' '/usr/include/*.h' '/usr/include/c++/*' '/usr/include/cc++/*' '/usr/include/omniORB4/*' '/usr/include/sigc++-2.0/sigc++/*'\
 	'/usr/include/catch.hpp' -d $(abs_srcdir) -o $(abs_srcdir)/.coverage.total
 	$(LCOV_GENHTML) --no-branch-coverage -o $(COVERAGE_REPORT_DIR) $(abs_srcdir)/.coverage.total
 	rm -f $(abs_srcdir)/.coverage.base $(abs_srcdir)/.coverage.run $(abs_srcdir)/.coverage.total