Add comment: cppcheck-suppress constVariablePointer

katono · katono · commit 1cdb45bf8104 · 2024-11-18T23:03:54.000+09:00
diff --git a/platform/PosixOSWrapper/PosixThreadFactory.cpp b/platform/PosixOSWrapper/PosixThreadFactory.cpp
@@ -35,7 +35,7 @@ void PosixThreadFactory::PosixThread::setPriority(int priority)
 	}
 
 	// INHERIT_PRIORITY
-	OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread();
+	OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread(); // cppcheck-suppress constVariablePointer
 	int this_priority;
 	if (t == nullptr) {
 		this_priority = OSWrapper::Thread::getNormalPriority();
diff --git a/platform/StdCppOSWrapper/StdCppThreadFactory.cpp b/platform/StdCppOSWrapper/StdCppThreadFactory.cpp
@@ -121,7 +121,7 @@ void StdCppThreadFactory::StdCppThread::setPriority(int priority)
 		return;
 	}
 	// INHERIT_PRIORITY
-	OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread();
+	OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread(); // cppcheck-suppress constVariablePointer
 	int this_priority;
 	if (t == nullptr) {
 		this_priority = OSWrapper::Thread::getNormalPriority();
diff --git a/platform/WindowsOSWrapper/WindowsThreadFactory.cpp b/platform/WindowsOSWrapper/WindowsThreadFactory.cpp
@@ -36,7 +36,7 @@ void WindowsThreadFactory::WindowsThread::setPriority(int priority)
 	}
 
 	// INHERIT_PRIORITY
-	OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread();
+	OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread(); // cppcheck-suppress constVariablePointer
 	int this_priority;
 	if (t == nullptr) {
 		this_priority = OSWrapper::Thread::getNormalPriority();

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ void PosixThreadFactory::PosixThread::setPriority(int priority)`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`// INHERIT_PRIORITY`
`38`		`- OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread();`
	`38`	`+ OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread(); // cppcheck-suppress constVariablePointer`
`39`	`39`	`int this_priority;`
`40`	`40`	`if (t == nullptr) {`
`41`	`41`	`this_priority = OSWrapper::Thread::getNormalPriority();`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ void StdCppThreadFactory::StdCppThread::setPriority(int priority)`
`121`	`121`	`return;`
`122`	`122`	`}`
`123`	`123`	`// INHERIT_PRIORITY`
`124`		`- OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread();`
	`124`	`+ OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread(); // cppcheck-suppress constVariablePointer`
`125`	`125`	`int this_priority;`
`126`	`126`	`if (t == nullptr) {`
`127`	`127`	`this_priority = OSWrapper::Thread::getNormalPriority();`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ void WindowsThreadFactory::WindowsThread::setPriority(int priority)`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`// INHERIT_PRIORITY`
`39`		`- OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread();`
	`39`	`+ OSWrapper::Thread* t = OSWrapper::Thread::getCurrentThread(); // cppcheck-suppress constVariablePointer`
`40`	`40`	`int this_priority;`
`41`	`41`	`if (t == nullptr) {`
`42`	`42`	`this_priority = OSWrapper::Thread::getNormalPriority();`