2025-11-18 00:53 ftp://ftp.de.debian.org/debian/pool/main/k/knetwalk/knetwalk_25.04.0.orig.tar.xz cppcheck-options: --library=posix --library=gnu --library=bsd --library=qt --library=kde --inconclusive --enable=style,information --inline-suppr --template=daca2 --disable=missingInclude --suppress=unmatchedSuppression --check-library --debug-warnings --suppress=autoNoType --suppress=valueFlowBailout --suppress=bailoutUninitVar --suppress=symbolDatabaseWarning --suppress=normalCheckLevelConditionExpressions -D__GNUC__ --platform=unix64 -j90 platform: Linux-6.8.0-87-generic-x86_64-with-glibc2.39 python: 3.12.3 client-version: 1.3.69 compiler: g++ (Ubuntu 13.3.0-6ubuntu2~24.04) 13.3.0 cppcheck: head 2.18.0 head-info: d8b6e08 (2025-11-17 23:40:03 +0100) count: 39 39 elapsed-time: 0.2 0.2 head-timing-info: old-timing-info: head results: knetwalk-25.04.0/src/abstractgrid.h:60:5: warning:inconclusive: Member variable 'Move::m_index' is not initialized in the constructor. [uninitMemberVar] knetwalk-25.04.0/src/abstractgrid.h:60:5: warning:inconclusive: Member variable 'Move::m_move' is not initialized in the constructor. [uninitMemberVar] knetwalk-25.04.0/autotests/abstractgridtest.cpp:16:14: warning: The class 'AbstractGridTester' defines member function with name 'initializeGrid' also defined in its parent class 'AbstractGrid'. [duplInheritedMember] knetwalk-25.04.0/src/abstractgrid.h:93:10: note: Parent function 'AbstractGrid::initializeGrid' knetwalk-25.04.0/autotests/abstractgridtest.cpp:16:14: note: Derived function 'AbstractGridTester::initializeGrid' knetwalk-25.04.0/src/main.cpp:47:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KNETWALK_VERSION_STRING [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/gameview.cpp:33:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SizeRootObjectToView [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/mainwindow.cpp:69:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable QVariant [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/mainwindow.cpp:116:58: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable gameRunningChanged [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/mainwindow.cpp:179:37: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable settingsChanged [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/mainwindow.cpp:186:60: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Name [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/mainwindow.cpp:199:50: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable VeryHard [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/mainwindow.cpp:238:37: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Score [valueFlowBailoutIncompleteVar] knetwalk-25.04.0/src/gameview.h:28:9: style:inconclusive: Either there is a missing 'override', or the member function 'GameView::minimumMoves' can be const. [functionConst] knetwalk-25.04.0/src/gameview.h:29:9: style:inconclusive: Either there is a missing 'override', or the member function 'GameView::cellCount' can be const. [functionConst] knetwalk-25.04.0/src/mainwindow.cpp:42:5: style: Class 'GeneralConfig' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] knetwalk-25.04.0/src/mainwindow.cpp:54:5: style: Class 'CustomGameConfig' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] knetwalk-25.04.0/src/abstractgrid.cpp:116:15: warning: Member variable 'AbstractGrid::m_isWrapped' is not initialized in the constructor. [uninitMemberVar] knetwalk-25.04.0/src/abstractgrid.cpp:116:15: warning: Member variable 'AbstractGrid::server_index' is not initialized in the constructor. [uninitMemberVar] knetwalk-25.04.0/src/abstractgrid.cpp:116:15: warning: Member variable 'AbstractGrid::m_minimumMoves' is not initialized in the constructor. [uninitMemberVar] knetwalk-25.04.0/src/abstractgrid.cpp:195:20: style: Unused private function: 'AbstractGrid::print' [unusedPrivateFunction] knetwalk-25.04.0/src/abstractgrid.h:102:10: note: Unused private function: 'AbstractGrid::print' knetwalk-25.04.0/src/abstractgrid.cpp:195:20: note: Unused private function: 'AbstractGrid::print' knetwalk-25.04.0/src/abstractgrid.cpp:351:26: performance:inconclusive: Technically the member function 'AbstractGrid::invertDirection' can be static (but you may consider moving to unnamed namespace). [functionStatic] knetwalk-25.04.0/src/abstractgrid.h:129:16: note: Technically the member function 'AbstractGrid::invertDirection' can be static (but you may consider moving to unnamed namespace). knetwalk-25.04.0/src/abstractgrid.cpp:351:26: note: Technically the member function 'AbstractGrid::invertDirection' can be static (but you may consider moving to unnamed namespace). knetwalk-25.04.0/src/abstractgrid.h:34:18: style: Virtual function 'makeEmpty' is called from constructor 'AbstractCell(int index)' at line 18. Dynamic binding is not used. [virtualCallInConstructor] knetwalk-25.04.0/src/abstractgrid.cpp:18:5: note: Calling makeEmpty knetwalk-25.04.0/src/abstractgrid.h:34:18: note: makeEmpty is a virtual function knetwalk-25.04.0/src/abstractgrid.cpp:83:53: style: Operator '|' with one operand equal to zero is redundant. [badBitmaskCheck] knetwalk-25.04.0/src/abstractgrid.cpp:94:53: style: Operator '|' with one operand equal to zero is redundant. [badBitmaskCheck] knetwalk-25.04.0/src/abstractgrid.cpp:440:14: style: The scope of the variable 'x' can be reduced. [variableScope] knetwalk-25.04.0/src/abstractgrid.cpp:441:14: style: The scope of the variable 'y' can be reduced. [variableScope] knetwalk-25.04.0/src/abstractgrid.cpp:126:69: style:inconclusive: Function 'initializeGrid' argument 3 names different: declaration 'w' definition 'wrapping'. [funcArgNamesDifferent] knetwalk-25.04.0/src/abstractgrid.h:93:59: note: Function 'initializeGrid' argument 3 names different: declaration 'w' definition 'wrapping'. knetwalk-25.04.0/src/abstractgrid.cpp:126:69: note: Function 'initializeGrid' argument 3 names different: declaration 'w' definition 'wrapping'. knetwalk-25.04.0/src/abstractgrid.cpp:367:24: style: Variable 'cell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:437:24: style: Variable 'cell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:521:24: style: Variable 'cell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:551:23: style: Variable 'cell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:552:23: style: Variable 'ucell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:553:23: style: Variable 'rcell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:554:23: style: Variable 'dcell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:555:23: style: Variable 'lcell' can be declared as pointer to const [constVariablePointer] knetwalk-25.04.0/src/abstractgrid.cpp:259:49: style: Consider using std::count_if algorithm instead of a raw loop. [useStlAlgorithm] knetwalk-25.04.0/src/abstractgrid.cpp:295:88: style: Consider using std::distance algorithm instead of a raw loop. [useStlAlgorithm] knetwalk-25.04.0/src/abstractgrid.cpp:368:36: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] knetwalk-25.04.0/src/abstractgrid.cpp:437:5: style: Consider using std::all_of or std::none_of algorithm instead of a raw loop. [useStlAlgorithm] knetwalk-25.04.0/src/abstractgrid.cpp:521:5: style: Consider using std::all_of or std::none_of algorithm instead of a raw loop. [useStlAlgorithm] diff: DONE