2022-06-26 18:51
ftp://ftp.de.debian.org/debian/pool/main/c/cohomcalg/cohomcalg_0.32+ds.orig.tar.gz
cppcheck-options: --library=posix --library=gnu  --showtime=top5 --check-library --inconclusive --enable=style,information --template=daca2 -D__GNUC__ --platform=unix64 -rp=temp -j1
platform: Linux-5.4.0-121-generic-x86_64-with-glibc2.29
python: 3.8.10
client-version: 1.3.24
compiler: g++ (Ubuntu 10.3.0-1ubuntu1~20.04) 10.3.0
cppcheck: head 2.8
head-info: 242afc3 (2022-06-25 21:42:55 +0200)
count: 66 66
elapsed-time: 4.3 4.2
head-timing-info:
 Tokenizer::simplifyTokens1: 1.27599s (avg. 0.255198s - 5 result(s))
 Tokenizer::simplifyTokens1::ValueFlow: 1.11114s (avg. 0.222229s - 5 result(s))
 Tokenizer::simplifyTokens1::createSymbolDatabase: 0.078164s (avg. 0.0156328s - 5 result(s))
 Tokenizer::createTokens: 0.024738s (avg. 0.0049476s - 5 result(s))
 Other::runChecks: 0.018642s (avg. 0.0037284s - 5 result(s))
 Overall time: 2.61017s

old-timing-info:
 Tokenizer::simplifyTokens1: 1.24418s (avg. 0.248836s - 5 result(s))
 Tokenizer::simplifyTokens1::ValueFlow: 1.08566s (avg. 0.217132s - 5 result(s))
 Tokenizer::simplifyTokens1::createSymbolDatabase: 0.072292s (avg. 0.0144584s - 5 result(s))
 Other::runChecks: 0.020998s (avg. 0.0041996s - 5 result(s))
 Tokenizer::createTokens: 0.020556s (avg. 0.0041112s - 5 result(s))
 Overall time: 2.54713s

head results:
cohomCalg-0.32/source/iohandler.cpp:59:10: performance:inconclusive: Technically the member function 'CInputFile::ParseMonomialFileCommand' can be static (but you may consider moving to unnamed namespace). [functionStatic]
cohomCalg-0.32/source/iohandler.cpp:461:18: note: Technically the member function 'CInputFile::ParseMonomialFileCommand' can be static (but you may consider moving to unnamed namespace).
cohomCalg-0.32/source/iohandler.cpp:59:10: note: Technically the member function 'CInputFile::ParseMonomialFileCommand' can be static (but you may consider moving to unnamed namespace).
cohomCalg-0.32/source/iohandler.cpp:60:7: performance:inconclusive: Technically the member function 'CInputFile::DiscardCommand' can be static (but you may consider moving to unnamed namespace). [functionStatic]
cohomCalg-0.32/source/iohandler.cpp:500:18: note: Technically the member function 'CInputFile::DiscardCommand' can be static (but you may consider moving to unnamed namespace).
cohomCalg-0.32/source/iohandler.cpp:60:7: note: Technically the member function 'CInputFile::DiscardCommand' can be static (but you may consider moving to unnamed namespace).
cohomCalg-0.32/source/iohandler.cpp:63:10: performance:inconclusive: Technically the member function 'CInputFile::HandleSyntaxError' can be static (but you may consider moving to unnamed namespace). [functionStatic]
cohomCalg-0.32/source/iohandler.cpp:160:18: note: Technically the member function 'CInputFile::HandleSyntaxError' can be static (but you may consider moving to unnamed namespace).
cohomCalg-0.32/source/iohandler.cpp:63:10: note: Technically the member function 'CInputFile::HandleSyntaxError' can be static (but you may consider moving to unnamed namespace).
cohomCalg-0.32/source/iohandler.cpp:1366:25: warning: %d in format string (no. 6) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint]
cohomCalg-0.32/source/iohandler.cpp:218:21: style: The scope of the variable 'dummy' can be reduced. [variableScope]
cohomCalg-0.32/source/iohandler.cpp:90:54: performance: Function parameter 'strFileName' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:133:47: performance: Function parameter 'strFileName' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:743:50: performance: Function parameter 'strFileName' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:743:70: performance: Function parameter 'strAppend' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:794:76: performance: Function parameter 'strSep' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:794:91: performance: Function parameter 'strZeroVal' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:1001:72: performance: Function parameter 'strFileName' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/iohandler.cpp:1044:71: performance: Function parameter 'strFileName' should be passed by const reference. [passedByValue]
cohomCalg-0.32/source/main.h:123:64: style: Parameter 'filename' can be declared as reference to const [constParameter]
cohomCalg-0.32/source/iohandler.cpp:676:35: error: Uninitialized variable: icd.GLSMch [uninitvar]
cohomCalg-0.32/source/iohandler.cpp:666:27: note: Assuming condition is false
cohomCalg-0.32/source/iohandler.cpp:676:35: note: Uninitialized variable: icd.GLSMch
cohomCalg-0.32/source/iohandler.cpp:547:18: style: Variable 'bResult' is assigned a value that is never used. [unreadVariable]
cohomCalg-0.32/source/iohandler.cpp:677:9: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/iohandler.cpp:1021:122: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/iohandler.cpp:1028:131: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/iohandler.cpp:1149:110: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/iohandler.cpp:1155:119: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/iohandler.cpp:1284:9: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/iohandler.cpp:1335:129: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/main.cpp:335:36: style: Condition 'bCheckForFilename' is always true [knownConditionTrueFalse]
cohomCalg-0.32/source/main.cpp:277:30: note: Assignment 'bCheckForFilename=true', assigned value is 1
cohomCalg-0.32/source/main.cpp:335:36: note: Condition 'bCheckForFilename' is always true
cohomCalg-0.32/source/main.cpp:242:63: style: Parameter 'argv' can be declared as const array [constParameter]
cohomCalg-0.32/source/main.cpp:535:33: style: Variable 'ts.t20SeqExactnessStart' is assigned a value that is never used. [unreadVariable]
cohomCalg-0.32/source/rationals.h:84:17: style: Unused private function: 'CRationals::PrintPolyLibConstraintMatrix' [unusedPrivateFunction]
cohomCalg-0.32/source/rationals.cpp:695:6: style: Condition '!WaitForAllWorkersFinish()' is always false [knownConditionTrueFalse]
cohomCalg-0.32/source/rationals.cpp:709:7: style: Condition '!WaitForAllWorkersFinish()' is always false [knownConditionTrueFalse]
cohomCalg-0.32/source/rationals.cpp:742:6: style: Condition '!WaitForAllWorkersFinish()' is always false [knownConditionTrueFalse]
cohomCalg-0.32/source/rationals.cpp:747:6: style: Condition '!WaitForAllWorkersFinish()' is always false [knownConditionTrueFalse]
cohomCalg-0.32/source/rationals.cpp:781:6: style: Condition '!WaitForAllWorkersFinish()' is always false [knownConditionTrueFalse]
cohomCalg-0.32/source/rationals.cpp:295:19: style: C-style pointer casting [cstyleCast]
cohomCalg-0.32/source/rationals.cpp:417:28: style: C-style pointer casting [cstyleCast]
cohomCalg-0.32/source/rationals.cpp:277:9: style: The scope of the variable 'pos' can be reduced. [variableScope]
cohomCalg-0.32/source/rationals.cpp:298:12: style: The scope of the variable 'tmp' can be reduced. [variableScope]
cohomCalg-0.32/source/rationals.cpp:298:17: style: The scope of the variable 'cureqn' can be reduced. [variableScope]
cohomCalg-0.32/source/rationals.cpp:298:25: style: The scope of the variable 'mathematica' can be reduced. [variableScope]
cohomCalg-0.32/source/rationals.cpp:277:13: style: Variable 'pos' is assigned a value that is never used. [unreadVariable]
cohomCalg-0.32/source/rationals.cpp:693:123: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:697:123: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:707:118: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:711:118: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:740:114: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:745:123: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:749:123: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:759:118: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:779:114: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:794:114: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/rationals.cpp:806:123: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:535:29: style: Condition 'seq_len<1' is always false [knownConditionTrueFalse]
cohomCalg-0.32/source/secondarycohom.cpp:534:56: note: Assignment 'seq_len=seq_end-seq_start+1', assigned value is greater than 0
cohomCalg-0.32/source/secondarycohom.cpp:535:29: note: Condition 'seq_len<1' is always false
cohomCalg-0.32/source/secondarycohom.cpp:525:49: style: Comparing expression of type 'unsigned int' against value 0. Condition is always true. [compareValueOutOfTypeRangeError]
cohomCalg-0.32/source/secondarycohom.cpp:287:27: style: C-style pointer casting [cstyleCast]
cohomCalg-0.32/source/secondarycohom.cpp:322:28: style: C-style pointer casting [cstyleCast]
cohomCalg-0.32/source/secondarycohom.cpp:288:20: style: The scope of the variable 'pbuf' can be reduced. [variableScope]
cohomCalg-0.32/source/secondarycohom.cpp:525:47: style: Unsigned expression 'i' can't be negative so it is unnecessary to test it. [unsignedPositive]
cohomCalg-0.32/source/secondarycohom.cpp:403:17: style: Local variable 'k' shadows outer variable [shadowVariable]
cohomCalg-0.32/source/secondarycohom.cpp:379:20: note: Shadowed declaration
cohomCalg-0.32/source/secondarycohom.cpp:403:17: note: Shadow variable
cohomCalg-0.32/source/secondarycohom.cpp:429:15: style: Variable 'stat' is assigned a value that is never used. [unreadVariable]
cohomCalg-0.32/source/secondarycohom.cpp:126:102: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:160:9: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:456:4: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:555:25: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:719:17: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:777:21: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/secondarycohom.cpp:782:5: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator]
cohomCalg-0.32/source/tokenizer.cpp:83:76: style: C-style pointer casting [cstyleCast]
diff:
head cohomCalg-0.32/source/main.cpp:242:63: style: Parameter 'argv' can be declared as const array [constParameter]
2.8 cohomCalg-0.32/source/main.cpp:242:63: style: Parameter 'argv' can be declared with const [constParameter]
head cohomCalg-0.32/source/main.h:123:64: style: Parameter 'filename' can be declared as reference to const [constParameter]
2.8 cohomCalg-0.32/source/main.h:123:64: style: Parameter 'filename' can be declared with const [constParameter]


DONE