2026-05-04 17:29 ftp://ftp.de.debian.org/debian/pool/main/r/r-cran-distory/r-cran-distory_1.4.5.orig.tar.gz cppcheck-options: --library=posix --library=gnu --library=bsd --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 -j1 platform: Linux-6.8.0-111-generic-x86_64-with-glibc2.39 python: 3.12.3 client-version: 1.3.70 compiler: g++ (Ubuntu 14.2.0-4ubuntu2~24.04.1) 14.2.0 cppcheck: head 2.20.0 head-info: 519958a (2026-05-04 15:03:10 +0200) count: 33 33 elapsed-time: 0.8 0.7 head-timing-info: old-timing-info: head results: distory/src/distory.c:18:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable FALSE [valueFlowBailoutIncompleteVar] distory/src/phylo.h:22:5: warning: Member variable 'PhyEdge::weight' is not initialized in the constructor. [uninitMemberVar] distory/src/phylo.h:22:5: warning: Member variable 'PhyEdge::id' is not initialized in the constructor. [uninitMemberVar] distory/src/phylo.h:22:5: style: Struct 'PhyEdge' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] distory/src/newick.cpp:136:13: style: Condition 'tpos>0' is always true [knownConditionTrueFalse] distory/src/newick.cpp:131:13: note: Assuming that condition 'tpos==0' is not redundant distory/src/newick.cpp:136:13: note: Condition 'tpos>0' is always true distory/src/phylo.h:58:39: style: Consider using std::count_if algorithm instead of a raw loop. [useStlAlgorithm] distory/src/phylo.h:84:39: style: Consider using std::count_if algorithm instead of a raw loop. [useStlAlgorithm] distory/src/phydist_r.cpp:297:9: style:inconclusive: Found duplicate branches for 'if' and 'else'. [duplicateBranch] distory/src/phydist_r.cpp:299:9: note: Found duplicate branches for 'if' and 'else'. distory/src/phydist_r.cpp:297:9: note: Found duplicate branches for 'if' and 'else'. distory/src/phydist_r.cpp:224:82: style:inconclusive: Function 'compute_phylo_distance_matrix' argument 3 names different: declaration 'm' definition 'distance_matrix'. [funcArgNamesDifferent] distory/src/phydist_r.cpp:29:82: note: Function 'compute_phylo_distance_matrix' argument 3 names different: declaration 'm' definition 'distance_matrix'. distory/src/phydist_r.cpp:224:82: note: Function 'compute_phylo_distance_matrix' argument 3 names different: declaration 'm' definition 'distance_matrix'. distory/src/phydist_r.cpp:202:43: style: Parameter 'trees_in' can be declared as reference to const [constParameterReference] distory/src/phydist_r.cpp:99:14: style: Variable 's1' can be declared as pointer to const [constVariablePointer] distory/src/phydist_r.cpp:101:14: style: Variable 's2' can be declared as pointer to const [constVariablePointer] distory/src/phydist_r.cpp:272:32: style: Parameter 'm' can be declared as pointer to const [constParameterPointer] distory/src/phydist_r.cpp:54:13: style: Consider using std::any_of algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:47:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable DBL_MIN [valueFlowBailoutIncompleteVar] distory/src/treedist-stripped.cpp:189:42: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable first [valueFlowBailoutIncompleteVar] distory/src/treedist-stripped.cpp:273:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable DBL_MAX [valueFlowBailoutIncompleteVar] distory/src/treedist-stripped.cpp:495:49: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable first [valueFlowBailoutIncompleteVar] distory/src/treedist-stripped.cpp:531:20: style: Local variable 'a' shadows outer argument [shadowArgument] distory/src/treedist-stripped.cpp:470:41: note: Shadowed declaration distory/src/treedist-stripped.cpp:531:20: note: Shadow variable distory/src/treedist-stripped.cpp:532:20: style: Local variable 'b' shadows outer argument [shadowArgument] distory/src/treedist-stripped.cpp:470:56: note: Shadowed declaration distory/src/treedist-stripped.cpp:532:20: note: Shadow variable distory/src/treedist-stripped.cpp:53:31: style: Parameter 'e1' can be declared as reference to const [constParameterReference] distory/src/treedist-stripped.cpp:53:44: style: Parameter 'e2' can be declared as reference to const [constParameterReference] distory/src/treedist-stripped.cpp:219:45: style: Parameter 'A_i' can be declared as reference to const [constParameterReference] distory/src/treedist-stripped.cpp:219:62: style: Parameter 'B_i' can be declared as reference to const [constParameterReference] distory/src/treedist-stripped.cpp:219:77: style: Parameter 'incompatible' can be declared as pointer to const [constParameterPointer] distory/src/treedist-stripped.cpp:105:13: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:228:19: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:232:19: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:331:26: style: Consider using std::copy_if algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:426:26: style: Consider using std::copy_if algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:537:15: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:545:15: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] distory/src/treedist-stripped.cpp:266:46: style: Unused variable: incompatible_edge_pairs [unusedVariable] diff: DONE