2025-05-05 06:20 ftp://ftp.de.debian.org/debian/pool/main/k/kbreakout/kbreakout_25.03.90.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 -j3 platform: Linux-6.8.0-1022-oracle-aarch64-with-glibc2.39 python: 3.12.3 client-version: 1.3.67 compiler: g++ (Ubuntu 13.3.0-6ubuntu2~24.04) 13.3.0 cppcheck: head 2.17.0 head-info: 88ef81e (2025-05-02 08:42:38 +0200) count: 31 31 elapsed-time: 0.1 0.1 head-timing-info: old-timing-info: head results: kbreakout-25.03.90/src/gameengine.cpp:54:25: performance:inconclusive: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. [functionStatic] kbreakout-25.03.90/src/gameengine.h:33:10: note: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. kbreakout-25.03.90/src/gameengine.cpp:54:25: note: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. kbreakout-25.03.90/src/gameengine.cpp:30:39: style:inconclusive: Function 'start' argument 1 names different: declaration 'levelset' definition 'l'. [funcArgNamesDifferent] kbreakout-25.03.90/src/gameengine.h:23:31: note: Function 'start' argument 1 names different: declaration 'levelset' definition 'l'. kbreakout-25.03.90/src/gameengine.cpp:30:39: note: Function 'start' argument 1 names different: declaration 'levelset' definition 'l'. kbreakout-25.03.90/src/canvaswidget.cpp:42:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable SizeRootObjectToView [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/canvaswidget.cpp:69:70: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable NativeText [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/canvaswidget.cpp:81:34: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Leave [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/canvaswidget.cpp:145:23: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable BlankCursor [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/main.cpp:43:74: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KBREAKOUT_VERSION_STRING [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/levelloader.cpp:58:51: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AppDataLocation [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/levelloader.cpp:101:20: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KBREAKOUT_General [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/levelloader.cpp:181:20: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable KBREAKOUT_General [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/levelloader.cpp:248:51: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable AppDataLocation [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:74:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Keys [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:111:59: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Key_Space [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:168:57: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Name [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:183:29: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable PrimaryAction [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:192:40: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Trigger [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:211:37: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Score [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:234:40: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Trigger [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:243:40: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Trigger [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:252:40: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Trigger [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/mainwindow.cpp:271:40: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable Trigger [valueFlowBailoutIncompleteVar] kbreakout-25.03.90/src/levelloader.cpp:21:14: warning: Member variable 'LevelLoader::m_oldstyle' is not initialized in the constructor. [uninitMemberVar] kbreakout-25.03.90/src/levelloader.cpp:21:14: warning: Member variable 'LevelLoader::m_lineNumber' is not initialized in the constructor. [uninitMemberVar] kbreakout-25.03.90/src/levelloader.h:25:13: performance: Function 'levelset()' should return member 'm_levelname' by const reference. [returnByReference] kbreakout-25.03.90/src/levelloader.cpp:119:17: style: Local variable 'level' shadows outer function [shadowFunction] kbreakout-25.03.90/src/levelloader.h:23:9: note: Shadowed declaration kbreakout-25.03.90/src/levelloader.cpp:119:17: note: Shadow variable kbreakout-25.03.90/src/levelloader.cpp:231:26: style: Variable 'nodeTextRead' is assigned a value that is never used. [unreadVariable] kbreakout-25.03.90/src/levelloader.cpp:234:26: style: Variable 'nodeTextRead' is assigned a value that is never used. [unreadVariable] kbreakout-25.03.90/src/mainwindow.cpp:221:5: warning: Either the condition 'ksdialog' is redundant or there is possible null pointer dereference: ksdialog. [nullPointerRedundantCheck] kbreakout-25.03.90/src/mainwindow.cpp:225:9: note: Assuming that condition 'ksdialog' is not redundant kbreakout-25.03.90/src/mainwindow.cpp:221:5: note: Null pointer dereference kbreakout-25.03.90/src/mainwindow.cpp:222:5: warning: Either the condition 'ksdialog' is redundant or there is possible null pointer dereference: ksdialog. [nullPointerRedundantCheck] kbreakout-25.03.90/src/mainwindow.cpp:225:9: note: Assuming that condition 'ksdialog' is not redundant kbreakout-25.03.90/src/mainwindow.cpp:222:5: note: Null pointer dereference kbreakout-25.03.90/src/mainwindow.cpp:223:5: warning: Either the condition 'ksdialog' is redundant or there is possible null pointer dereference: ksdialog. [nullPointerRedundantCheck] kbreakout-25.03.90/src/mainwindow.cpp:225:9: note: Assuming that condition 'ksdialog' is not redundant kbreakout-25.03.90/src/mainwindow.cpp:223:5: note: Null pointer dereference kbreakout-25.03.90/src/mainwindow.cpp:296:17: style: The comparison 'dontAsk == false' is always true because 'dontAsk' and 'false' represent the same value. [knownConditionTrueFalse] kbreakout-25.03.90/src/mainwindow.cpp:286:20: note: 'dontAsk' is assigned value 'false' here. kbreakout-25.03.90/src/mainwindow.cpp:296:17: note: The comparison 'dontAsk == false' is always true because 'dontAsk' and 'false' represent the same value. diff: head kbreakout-25.03.90/src/gameengine.cpp:54:25: performance:inconclusive: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. [functionStatic] kbreakout-25.03.90/src/gameengine.h:33:10: note: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. kbreakout-25.03.90/src/gameengine.cpp:54:25: note: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. 2.17.0 kbreakout-25.03.90/src/gameengine.h:33:10: performance:inconclusive: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. [functionStatic] kbreakout-25.03.90/src/gameengine.cpp:54:25: note: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. kbreakout-25.03.90/src/gameengine.h:33:10: note: Either there is a missing 'override', or the member function 'GameEngine::deleteAllObjects' can be static. DONE