more simplify Pointer

libs/main/thread/piprotectedvariable.h

@@ -49,15 +49,11 @@ public:
 	public:
 		//! \~english Move constructor - transfers ownership of the lock.
 		//! \~russian Конструктор перемещения - передает владение блокировкой.
-		Pointer(Pointer && other): pv(other.pv) { other.can_unlock = false; };
+		Pointer(Pointer && other): pv(other.pv) { pv.mutex.lock(); };
 
 		//! \~english Destroys wrapper and releases the mutex.
 		//! \~russian Уничтожает обертку и освобождает мьютекс.
-		~Pointer() {
+		~Pointer() { pv.mutex.unlock(); }
-			if (can_unlock) {
-				pv.mutex.unlock();
-			}
-		}
 
 		//! \~english Returns pointer access to the protected value.
 		//! \~russian Возвращает указательный доступ к защищенному значению.
@@ -72,7 +68,6 @@ public:
 		explicit Pointer(PIProtectedVariable<T> & v): pv(v) { pv.mutex.lock(); }
 
 		PIProtectedVariable<T> & pv;
-		bool can_unlock = true;
 	};
 
 	//! \~english Replaces the protected value with \a v.

tests/thread/piprotectedvariable_test.cpp

@@ -82,7 +82,9 @@ TEST(PIProtectedVariable_ThreadSafety, ConcurrentReadWrite) {
 			for (int j = 0; j < NUM_ITERATIONS; ++j) {
 				auto val = pv.getRef();
 				(*val)++;
+				auto val2 = pv.getRef();
 				writeCount++;
+				ASSERT_EQ(writeCount, *val2);
 			}
 		}));
 	}