2024-04-24 14:12 ftp://ftp.de.debian.org/debian/pool/main/f/fastp/fastp_0.23.4+dfsg.orig.tar.xz 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 -D__GNUC__ --platform=unix64 -j1 platform: Linux-5.15.0-105-generic-x86_64-with-glibc2.35 python: 3.10.12 client-version: 1.3.56 compiler: g++ (Ubuntu 12.3.0-1ubuntu1~22.04) 12.3.0 cppcheck: head 2.14.0 head-info: 9fab9b9 (2024-04-24 10:46:48 +0200) count: 486 486 elapsed-time: 18.6 18.5 head-timing-info: old-timing-info: head results: fastp-0.23.4/src/adaptertrimmer.cpp:28:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/adaptertrimmer.cpp:64:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/adaptertrimmer.cpp:156:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/options.h:199:5: warning: Member variable 'AdapterOptions::hasFasta' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/options.h:246:5: warning: Member variable 'QualityFilteringOptions::avgQualReq' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/options.h:111:9: performance: Variable 'delimiter' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/sequence.h:14:5: style: Class 'Sequence' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/filterresult.h:29:5: style: Class 'FilterResult' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/adaptertrimmer.cpp:154:13: style: Redundant initialization for 'trimmed'. The initialized value is overwritten before it is read. [redundantInitialization] fastp-0.23.4/src/adaptertrimmer.cpp:141:18: note: trimmed is initialized fastp-0.23.4/src/adaptertrimmer.cpp:154:13: note: trimmed is overwritten fastp-0.23.4/src/adaptertrimmer.cpp:16:9: style: The scope of the variable 'ol' can be reduced. [variableScope] fastp-0.23.4/src/adaptertrimmer.cpp:46:49: style:inconclusive: Function 'trimByMultiSequences' argument 1 names different: declaration 'r1' definition 'r'. [funcArgNamesDifferent] fastp-0.23.4/src/adaptertrimmer.h:21:44: note: Function 'trimByMultiSequences' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/adaptertrimmer.cpp:46:49: note: Function 'trimByMultiSequences' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/adaptertrimmer.cpp:70:43: style:inconclusive: Function 'trimBySequence' argument 1 names different: declaration 'r1' definition 'r'. [funcArgNamesDifferent] fastp-0.23.4/src/adaptertrimmer.h:20:38: note: Function 'trimBySequence' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/adaptertrimmer.cpp:70:43: note: Function 'trimBySequence' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/adaptertrimmer.cpp:70:72: style:inconclusive: Function 'trimBySequence' argument 3 names different: declaration 'adapter' definition 'adapterseq'. [funcArgNamesDifferent] fastp-0.23.4/src/adaptertrimmer.h:20:68: note: Function 'trimBySequence' argument 3 names different: declaration 'adapter' definition 'adapterseq'. fastp-0.23.4/src/adaptertrimmer.cpp:70:72: note: Function 'trimBySequence' argument 3 names different: declaration 'adapter' definition 'adapterseq'. fastp-0.23.4/src/adaptertrimmer.cpp:70:72: style: Parameter 'adapterseq' can be declared as reference to const [constParameterReference] fastp-0.23.4/src/adaptertrimmer.cpp:54:13: style: Variable 'originalSeq' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/adaptertrimmer.cpp:154:13: style: Variable 'trimmed' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/util.h:55:24: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable npos [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:78:33: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable npos [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:104:27: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable npos [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:128:24: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable npos [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:138:24: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable npos [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:187:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:203:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:271:5: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:280:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable stderr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/basecorrector.cpp:70:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/util.h:279:12: portability: Non reentrant function 'localtime' called. For threadsafe applications it is recommended to use the reentrant replacement function 'localtime_r'. [prohibitedlocaltimeCalled] fastp-0.23.4/src/util.h:253:46: style: C-style pointer casting [cstyleCast] fastp-0.23.4/src/util.h:257:46: style: C-style pointer casting [cstyleCast] fastp-0.23.4/src/util.h:276:34: performance: Function parameter 's' should be passed by const reference. [passedByValue] fastp-0.23.4/src/util.h:41:34: style: Parameter 'value' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/util.h:279:9: style: Variable 't' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/duplicate.cpp:56:31: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable atomic_uchar [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/duplicate.h:21:10: style:inconclusive: Technically the member function 'Duplicate::seq2intvector' can be const. [functionConst] fastp-0.23.4/src/duplicate.cpp:91:17: note: Technically the member function 'Duplicate::seq2intvector' can be const. fastp-0.23.4/src/duplicate.h:21:10: note: Technically the member function 'Duplicate::seq2intvector' can be const. fastp-0.23.4/src/duplicate.h:23:12: style:inconclusive: Technically the member function 'Duplicate::getDupRate' can be const. [functionConst] fastp-0.23.4/src/duplicate.cpp:169:19: note: Technically the member function 'Duplicate::getDupRate' can be const. fastp-0.23.4/src/duplicate.h:23:12: note: Technically the member function 'Duplicate::getDupRate' can be const. fastp-0.23.4/src/duplicate.cpp:52:5: warning: Class 'Duplicate' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/duplicate.cpp:52:5: warning: Class 'Duplicate' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/duplicate.h:16:5: style: Class 'Duplicate' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/duplicate.cpp:118:33: style:inconclusive: Function 'checkRead' argument 1 names different: declaration 'r1' definition 'r'. [funcArgNamesDifferent] fastp-0.23.4/src/duplicate.h:19:26: note: Function 'checkRead' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/duplicate.cpp:118:33: note: Function 'checkRead' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/duplicate.cpp:154:42: style: Parameter 'positions' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/singleproducersingleconsumerlist.h:96:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/singleproducersingleconsumerlist.h:135:30: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/evaluator.cpp:401:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/evaluator.cpp:620:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/evaluator.cpp:722:5: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/singleproducersingleconsumerlist.h:49:9: performance: Variable 'value' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/singleproducersingleconsumerlist.h:50:9: performance: Variable 'nextItemReady' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/singleproducersingleconsumerlist.h:55:9: performance: Variable 'nextItemReady' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/singleproducersingleconsumerlist.h:68:9: performance: Variable 'producerFinished' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/singleproducersingleconsumerlist.h:69:9: performance: Variable 'consumerFinished' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/evaluator.h:24:10: performance:inconclusive: Technically the member function 'Evaluator::computeOverRepSeq' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/evaluator.cpp:65:17: note: Technically the member function 'Evaluator::computeOverRepSeq' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:24:10: note: Technically the member function 'Evaluator::computeOverRepSeq' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:25:9: performance:inconclusive: Technically the member function 'Evaluator::computeSeqLen' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/evaluator.cpp:41:16: note: Technically the member function 'Evaluator::computeSeqLen' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:25:9: note: Technically the member function 'Evaluator::computeSeqLen' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:31:12: performance:inconclusive: Technically the member function 'Evaluator::int2seq' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/evaluator.cpp:652:19: note: Technically the member function 'Evaluator::int2seq' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:31:12: note: Technically the member function 'Evaluator::int2seq' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:33:9: performance:inconclusive: Technically the member function 'Evaluator::seq2int' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/evaluator.cpp:668:16: note: Technically the member function 'Evaluator::seq2int' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/evaluator.h:33:9: note: Technically the member function 'Evaluator::seq2int' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/singleproducersingleconsumerlist.h:87:19: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList::size' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:90:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList::canBeConsumed' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:116:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList::isProducerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:119:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList::isConsumerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/evaluator.h:15:5: style: Class 'Evaluator' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/singleproducersingleconsumerlist.h:48:12: style: Struct 'LockFreeListItem' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/readpool.h:18:5: style: Class 'ReadPool' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/fastqreader.h:39:2: style: Class 'FastqReader' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/nucleotidetree.h:27:5: style: Class 'NucleotideTree' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/evaluator.cpp:45:10: style: The scope of the variable 'reachedEOF' can be reduced. [variableScope] fastp-0.23.4/src/evaluator.cpp:73:10: style: The scope of the variable 'reachedEOF' can be reduced. [variableScope] fastp-0.23.4/src/evaluator.cpp:65:84: style:inconclusive: Function 'computeOverRepSeq' argument 3 names different: declaration 'seqLen' definition 'seqlen'. [funcArgNamesDifferent] fastp-0.23.4/src/evaluator.h:24:77: note: Function 'computeOverRepSeq' argument 3 names different: declaration 'seqLen' definition 'seqlen'. fastp-0.23.4/src/evaluator.cpp:65:84: note: Function 'computeOverRepSeq' argument 3 names different: declaration 'seqLen' definition 'seqlen'. fastp-0.23.4/src/evaluator.cpp:664:50: style:inconclusive: Function 'seq2int' argument 3 names different: declaration 'seqlen' definition 'keylen'. [funcArgNamesDifferent] fastp-0.23.4/src/evaluator.h:32:43: note: Function 'seq2int' argument 3 names different: declaration 'seqlen' definition 'keylen'. fastp-0.23.4/src/evaluator.cpp:664:50: note: Function 'seq2int' argument 3 names different: declaration 'seqlen' definition 'keylen'. fastp-0.23.4/src/evaluator.cpp:668:50: style:inconclusive: Function 'seq2int' argument 3 names different: declaration 'seqlen' definition 'keylen'. [funcArgNamesDifferent] fastp-0.23.4/src/evaluator.h:33:43: note: Function 'seq2int' argument 3 names different: declaration 'seqlen' definition 'keylen'. fastp-0.23.4/src/evaluator.cpp:668:50: note: Function 'seq2int' argument 3 names different: declaration 'seqlen' definition 'keylen'. fastp-0.23.4/src/singleproducersingleconsumerlist.h:133:16: style: Local variable 'size' shadows outer function [shadowFunction] fastp-0.23.4/src/singleproducersingleconsumerlist.h:87:19: note: Shadowed declaration fastp-0.23.4/src/singleproducersingleconsumerlist.h:133:16: note: Shadow variable fastp-0.23.4/src/evaluator.cpp:324:14: style: Local variable 'bases' shadows outer variable [shadowVariable] fastp-0.23.4/src/evaluator.cpp:214:10: note: Shadowed declaration fastp-0.23.4/src/evaluator.cpp:324:14: note: Shadow variable fastp-0.23.4/src/evaluator.cpp:41:37: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/evaluator.cpp:65:42: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/evaluator.cpp:632:44: performance: Function parameter 'seq' should be passed by const reference. [passedByValue] fastp-0.23.4/src/evaluator.cpp:668:32: style: Parameter 'seq' can be declared as reference to const [constParameterReference] fastp-0.23.4/src/evaluator.cpp:86:13: style: Variable 'steps' can be declared as const array [constVariable] fastp-0.23.4/src/evaluator.cpp:324:14: style: Variable 'bases' can be declared as const array [constVariable] fastp-0.23.4/src/evaluator.cpp:653:10: style: Variable 'bases' can be declared as const array [constVariable] fastp-0.23.4/src/evaluator.cpp:45:21: style: Variable 'reachedEOF' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:52:24: style: Variable 'reachedEOF' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:73:21: style: Variable 'reachedEOF' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:78:24: style: Variable 'reachedEOF' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:478:26: style: Variable 'data' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:584:26: style: Variable 'data' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:600:26: style: Variable 'data' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:637:16: style: Variable 'desc' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:669:9: style: Variable 'rlen' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/evaluator.cpp:103:58: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:137:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:148:13: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:322:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:344:66: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:346:70: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:387:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/evaluator.cpp:635:68: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/fastareader.cpp:11:15: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable LC_ALL [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/fastareader.h:25:19: style:inconclusive: Technically the member function 'FastaReader::currentID' can be const. [functionConst] fastp-0.23.4/src/fastareader.h:30:19: style:inconclusive: Technically the member function 'FastaReader::currentDescription' can be const. [functionConst] fastp-0.23.4/src/fastareader.h:35:19: style:inconclusive: Technically the member function 'FastaReader::currentSequence' can be const. [functionConst] fastp-0.23.4/src/fastareader.h:19:5: style: Class 'FastaReader' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/fastareader.h:25:19: performance: Function 'currentID()' should return member 'mCurrentID' by const reference. [returnByReference] fastp-0.23.4/src/fastareader.h:30:19: performance: Function 'currentDescription()' should return member 'mCurrentDescription' by const reference. [returnByReference] fastp-0.23.4/src/fastareader.h:35:19: performance: Function 'currentSequence()' should return member 'mCurrentSequence' by const reference. [returnByReference] fastp-0.23.4/src/fastareader.cpp:6:33: style:inconclusive: Function 'FastaReader' argument 1 names different: declaration 'fastaFile' definition 'faFile'. [funcArgNamesDifferent] fastp-0.23.4/src/fastareader.h:19:24: note: Function 'FastaReader' argument 1 names different: declaration 'fastaFile' definition 'faFile'. fastp-0.23.4/src/fastareader.cpp:6:33: note: Function 'FastaReader' argument 1 names different: declaration 'fastaFile' definition 'faFile'. fastp-0.23.4/src/fastareader.cpp:6:33: performance: Function parameter 'faFile' should be passed by const reference. [passedByValue] fastp-0.23.4/src/fastqreader.cpp:178:12: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable stdin [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/fastqreader.cpp:321:3: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/fastqreader.cpp:0:0: debug: ValueFlow maximum iterations exceeded [valueFlowMaxIterations] fastp-0.23.4/src/fastqreader.cpp:398:18: warning: Member variable 'FastqReaderPair::mInterleaved' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/fastqreader.h:61:7: style: Unused private function: 'FastqReader::clearLineBreaks' [unusedPrivateFunction] fastp-0.23.4/src/fastqreader.cpp:35:2: performance: Variable 'mFilename' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/fastqreader.h:41:7: style:inconclusive: Technically the member function 'FastqReader::isZipped' can be const. [functionConst] fastp-0.23.4/src/fastqreader.cpp:374:19: note: Technically the member function 'FastqReader::isZipped' can be const. fastp-0.23.4/src/fastqreader.h:41:7: note: Technically the member function 'FastqReader::isZipped' can be const. fastp-0.23.4/src/fastqreader.h:49:7: style:inconclusive: Technically the member function 'FastqReader::hasNoLineBreakAtEnd' can be const. [functionConst] fastp-0.23.4/src/fastqreader.cpp:62:19: note: Technically the member function 'FastqReader::hasNoLineBreakAtEnd' can be const. fastp-0.23.4/src/fastqreader.h:49:7: note: Technically the member function 'FastqReader::hasNoLineBreakAtEnd' can be const. fastp-0.23.4/src/fastqreader.h:61:7: performance:inconclusive: Technically the member function 'FastqReader::clearLineBreaks' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/fastqreader.cpp:202:19: note: Technically the member function 'FastqReader::clearLineBreaks' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/fastqreader.h:61:7: note: Technically the member function 'FastqReader::clearLineBreaks' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/fastqreader.cpp:44:2: warning: Class 'FastqReader' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/fastqreader.cpp:44:2: warning: Class 'FastqReader' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/fastqreader.cpp:405:2: warning: Class 'FastqReaderPair' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/fastqreader.cpp:405:2: warning: Class 'FastqReaderPair' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/fastqreader.cpp:46:22: style: C-style pointer casting [cstyleCast] fastp-0.23.4/src/fastqreader.cpp:50:23: style: Variable 'mHasNoLineBreakAtEnd' is reassigned a value before the old one has been used. [redundantAssignment] fastp-0.23.4/src/fastqreader.cpp:42:23: note: mHasNoLineBreakAtEnd is assigned fastp-0.23.4/src/fastqreader.cpp:50:23: note: mHasNoLineBreakAtEnd is overwritten fastp-0.23.4/src/fastqreader.cpp:34:33: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/fastqreader.cpp:348:37: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/fastqreader.cpp:361:34: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/fastqreader.cpp:403:41: performance: Function parameter 'leftName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/fastqreader.cpp:403:58: performance: Function parameter 'rightName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/fastqreader.cpp:220:13: style: Variable 'copied' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/singleproducersingleconsumerlist.h:53:12: warning: Member variable 'LockFreeListItem < ReadPack * >::value' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/threadconfig.h:34:9: style:inconclusive: Technically the member function 'ThreadConfig::getThreadId' can be const. [functionConst] fastp-0.23.4/src/filter.h:18:10: style:inconclusive: Technically the member function 'Filter::passLowComplexityFilter' can be const. [functionConst] fastp-0.23.4/src/filter.cpp:67:14: note: Technically the member function 'Filter::passLowComplexityFilter' can be const. fastp-0.23.4/src/filter.h:18:10: note: Technically the member function 'Filter::passLowComplexityFilter' can be const. fastp-0.23.4/src/filter.h:25:10: performance:inconclusive: Technically the member function 'Filter::match' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/filter.cpp:242:14: note: Technically the member function 'Filter::match' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/filter.h:25:10: note: Technically the member function 'Filter::match' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/writerthread.h:31:10: style:inconclusive: Technically the member function 'WriterThread::bufferLength' can be const. [functionConst] fastp-0.23.4/src/writerthread.h:32:12: style:inconclusive: Technically the member function 'WriterThread::getFilename' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:87:19: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < ReadPack * >::size' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:90:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < ReadPack * >::canBeConsumed' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:116:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < ReadPack * >::isProducerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:119:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < ReadPack * >::isConsumerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList < ReadPack * >' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList < ReadPack * >' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/processor.h:13:5: style: Class 'Processor' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/stats.h:17:5: style: Class 'Stats' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/filter.h:15:5: style: Class 'Filter' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/umiprocessor.h:14:5: style: Class 'UmiProcessor' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/peprocessor.h:28:5: style: Class 'PairEndProcessor' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/seprocessor.h:27:5: style: Class 'SingleEndProcessor' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/singleproducersingleconsumerlist.h:48:12: style: Struct 'LockFreeListItem < ReadPack * >' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/writerthread.h:32:12: performance: Function 'getFilename()' should return member 'mFilename' by const reference. [returnByReference] fastp-0.23.4/src/filter.cpp:242:49: performance: Function parameter 'target' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filter.cpp:242:36: style: Parameter 'list' can be declared as reference to const [constParameterReference] fastp-0.23.4/src/filterresult.cpp:170:5: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/filterresult.cpp:196:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/filterresult.cpp:251:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/filterresult.cpp:277:66: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/filterresult.cpp:288:30: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/filterresult.h:40:10: style:inconclusive: Technically the member function 'FilterResult::getTotalPolyXTrimmedReads' can be const. [functionConst] fastp-0.23.4/src/filterresult.cpp:154:20: note: Technically the member function 'FilterResult::getTotalPolyXTrimmedReads' can be const. fastp-0.23.4/src/filterresult.h:40:10: note: Technically the member function 'FilterResult::getTotalPolyXTrimmedReads' can be const. fastp-0.23.4/src/filterresult.h:41:10: style:inconclusive: Technically the member function 'FilterResult::getTotalPolyXTrimmedBases' can be const. [functionConst] fastp-0.23.4/src/filterresult.cpp:161:20: note: Technically the member function 'FilterResult::getTotalPolyXTrimmedBases' can be const. fastp-0.23.4/src/filterresult.h:41:10: note: Technically the member function 'FilterResult::getTotalPolyXTrimmedBases' can be const. fastp-0.23.4/src/filterresult.h:52:10: performance:inconclusive: Technically the member function 'FilterResult::outputAdaptersJson' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/filterresult.cpp:213:20: note: Technically the member function 'FilterResult::outputAdaptersJson' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/filterresult.h:52:10: note: Technically the member function 'FilterResult::outputAdaptersJson' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/filterresult.h:56:10: style:inconclusive: Technically the member function 'FilterResult::getTotalCorrectedBases' can be const. [functionConst] fastp-0.23.4/src/filterresult.cpp:88:20: note: Technically the member function 'FilterResult::getTotalCorrectedBases' can be const. fastp-0.23.4/src/filterresult.h:56:10: note: Technically the member function 'FilterResult::getTotalCorrectedBases' can be const. fastp-0.23.4/src/filterresult.h:58:10: style:inconclusive: Technically the member function 'FilterResult::getCorrectionNum' can be const. [functionConst] fastp-0.23.4/src/filterresult.cpp:106:20: note: Technically the member function 'FilterResult::getCorrectionNum' can be const. fastp-0.23.4/src/filterresult.h:58:10: note: Technically the member function 'FilterResult::getCorrectionNum' can be const. fastp-0.23.4/src/filterresult.cpp:16:5: warning: Class 'FilterResult' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/filterresult.cpp:16:5: warning: Class 'FilterResult' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/htmlreporter.h:20:5: style: Class 'HtmlReporter' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/filterresult.cpp:22:5: error: Mismatching allocation and deallocation: FilterResult::mCorrectionMatrix [mismatchAllocDealloc] fastp-0.23.4/src/filterresult.cpp:112:45: performance: Function parameter 'adapter' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:131:45: performance: Function parameter 'adapter1' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:131:62: performance: Function parameter 'adapter2' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:195:53: performance: Function parameter 'padding' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:250:60: performance: Function parameter 'padding' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:276:48: performance: Function parameter 'pad' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:276:60: performance: Function parameter 'key' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:287:62: performance: Function parameter 'padding' should be passed by const reference. [passedByValue] fastp-0.23.4/src/filterresult.cpp:213:84: style: Parameter 'adapterCounts' can be declared as reference to const [constParameterReference] fastp-0.23.4/src/filterresult.cpp:43:15: style: Variable 'current' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/filterresult.cpp:75:15: style: Variable 'current' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/filterresult.cpp:276:84: style: Parameter 'counts' can be declared as const array [constParameterReference] fastp-0.23.4/src/filterresult.cpp:218:15: style: Consider using std::accumulate algorithm instead of a raw loop. [useStlAlgorithm] fastp-0.23.4/src/filterresult.cpp:58:82: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/filterresult.cpp:64:82: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/filterresult.cpp:217:66: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/filterresult.cpp:228:66: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/filterresult.cpp:355:66: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/filterresult.cpp:377:66: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/htmlreporter.cpp:103:12: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:241:53: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:270:53: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:327:44: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable out [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:380:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:407:82: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:434:26: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/htmlreporter.cpp:425:20: portability: Non reentrant function 'localtime' called. For threadsafe applications it is recommended to use the reentrant replacement function 'localtime_r'. [prohibitedlocaltimeCalled] fastp-0.23.4/src/htmlreporter.cpp:7:15: warning: Member variable 'HtmlReporter::mDupMeanGC' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/htmlreporter.cpp:7:15: warning: Member variable 'HtmlReporter::mInsertHist' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/htmlreporter.cpp:7:15: warning: Member variable 'HtmlReporter::mInsertSizePeak' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/htmlreporter.h:36:10: style: Unused private function: 'HtmlReporter::reportDuplication' [unusedPrivateFunction] fastp-0.23.4/src/htmlreporter.h:31:18: performance:inconclusive: Technically the member function 'HtmlReporter::getCurrentSystemTime' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/htmlreporter.cpp:422:28: note: Technically the member function 'HtmlReporter::getCurrentSystemTime' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/htmlreporter.h:31:18: note: Technically the member function 'HtmlReporter::getCurrentSystemTime' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/htmlreporter.h:33:10: performance:inconclusive: Technically the member function 'HtmlReporter::printCSS' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/htmlreporter.cpp:379:20: note: Technically the member function 'HtmlReporter::printCSS' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/htmlreporter.h:33:10: note: Technically the member function 'HtmlReporter::printCSS' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/htmlreporter.h:34:10: performance:inconclusive: Technically the member function 'HtmlReporter::printJS' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/htmlreporter.cpp:406:20: note: Technically the member function 'HtmlReporter::printJS' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/htmlreporter.h:34:10: note: Technically the member function 'HtmlReporter::printJS' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/htmlreporter.cpp:56:28: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/htmlreporter.cpp:337:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:31: note: Calling function 'printSummary', 3rd argument 'preStats1' value is 0 fastp-0.23.4/src/htmlreporter.cpp:56:28: note: Null pointer dereference fastp-0.23.4/src/htmlreporter.cpp:60:28: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/htmlreporter.cpp:337:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:31: note: Calling function 'printSummary', 3rd argument 'preStats1' value is 0 fastp-0.23.4/src/htmlreporter.cpp:60:28: note: Null pointer dereference fastp-0.23.4/src/htmlreporter.cpp:64:26: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/htmlreporter.cpp:337:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:31: note: Calling function 'printSummary', 3rd argument 'preStats1' value is 0 fastp-0.23.4/src/htmlreporter.cpp:64:26: note: Null pointer dereference fastp-0.23.4/src/htmlreporter.cpp:68:26: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/htmlreporter.cpp:337:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:31: note: Calling function 'printSummary', 3rd argument 'preStats1' value is 0 fastp-0.23.4/src/htmlreporter.cpp:68:26: note: Null pointer dereference fastp-0.23.4/src/htmlreporter.cpp:72:25: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/htmlreporter.cpp:337:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:31: note: Calling function 'printSummary', 3rd argument 'preStats1' value is 0 fastp-0.23.4/src/htmlreporter.cpp:72:25: note: Null pointer dereference fastp-0.23.4/src/htmlreporter.cpp:35:12: style: The scope of the variable 'unit' can be reduced. [variableScope] fastp-0.23.4/src/htmlreporter.cpp:25:62: style:inconclusive: Function 'outputRow' argument 3 names different: declaration 'value' definition 'v'. [funcArgNamesDifferent] fastp-0.23.4/src/htmlreporter.h:26:59: note: Function 'outputRow' argument 3 names different: declaration 'value' definition 'v'. fastp-0.23.4/src/htmlreporter.cpp:25:62: note: Function 'outputRow' argument 3 names different: declaration 'value' definition 'v'. fastp-0.23.4/src/htmlreporter.cpp:29:64: style:inconclusive: Function 'outputRow' argument 3 names different: declaration 'value' definition 'v'. [funcArgNamesDifferent] fastp-0.23.4/src/htmlreporter.h:27:61: note: Function 'outputRow' argument 3 names different: declaration 'value' definition 'v'. fastp-0.23.4/src/htmlreporter.cpp:29:64: note: Function 'outputRow' argument 3 names different: declaration 'value' definition 'v'. fastp-0.23.4/src/htmlreporter.cpp:25:52: performance: Function parameter 'key' should be passed by const reference. [passedByValue] fastp-0.23.4/src/htmlreporter.cpp:29:52: performance: Function parameter 'key' should be passed by const reference. [passedByValue] fastp-0.23.4/src/htmlreporter.cpp:29:64: performance: Function parameter 'v' should be passed by const reference. [passedByValue] fastp-0.23.4/src/htmlreporter.cpp:35:12: style: Variable 'unit' can be declared as const array [constVariable] fastp-0.23.4/src/htmlreporter.cpp:425:14: style: Variable 'ptm' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/jsonreporter.cpp:24:44: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable out [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/jsonreporter.cpp:3:15: warning: Member variable 'JsonReporter::mDupMeanGC' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/jsonreporter.cpp:3:15: warning: Member variable 'JsonReporter::mInsertHist' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/jsonreporter.cpp:3:15: warning: Member variable 'JsonReporter::mInsertSizePeak' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/jsonreporter.h:20:5: style: Class 'JsonReporter' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/jsonreporter.cpp:35:28: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:146:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:35:28: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:39:28: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:146:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:39:28: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:43:26: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:146:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:43:26: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:47:26: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:146:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:47:26: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:51:25: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:146:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:51:25: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:55:29: warning: Either the condition 'postStats1' is redundant or there is possible null pointer dereference: postStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:156:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:55:29: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:59:29: warning: Either the condition 'postStats1' is redundant or there is possible null pointer dereference: postStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:156:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:59:29: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:63:27: warning: Either the condition 'postStats1' is redundant or there is possible null pointer dereference: postStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:156:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:63:27: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:67:27: warning: Either the condition 'postStats1' is redundant or there is possible null pointer dereference: postStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:156:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:67:27: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:71:26: warning: Either the condition 'postStats1' is redundant or there is possible null pointer dereference: postStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:156:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:71:26: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:86:52: warning: Either the condition 'preStats1' is redundant or there is possible null pointer dereference: preStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:146:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:86:52: note: Null pointer dereference fastp-0.23.4/src/jsonreporter.cpp:99:52: warning: Either the condition 'postStats1' is redundant or there is possible null pointer dereference: postStats1. [nullPointerRedundantCheck] fastp-0.23.4/src/jsonreporter.cpp:156:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/jsonreporter.cpp:99:52: note: Null pointer dereference fastp-0.23.4/src/main.cpp:34:9: debug: Failed to instantiate template "add". The checking continues anyway. [templateInstantiation] fastp-0.23.4/src/cmdline.h:148:20: debug: Failed to instantiate template "lexical_cast". The checking continues anyway. [templateInstantiation] fastp-0.23.4/src/cmdline.h:102:74: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable value [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:167:23: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:186:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:194:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:203:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:213:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:224:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:236:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:249:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:263:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:278:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:294:16: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable T [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:411:12: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cout [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:552:28: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:564:38: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:591:12: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/main.cpp:19:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/cmdline.h:329:8: performance:inconclusive: Technically the member function 'cmdline::parser::add < string >' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/cmdline.h:329:8: note: Technically the member function 'cmdline::parser::add < string >' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/cmdline.h:329:8: note: Technically the member function 'cmdline::parser::add < string >' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/cmdline.h:329:8: performance:inconclusive: Technically the member function 'cmdline::parser::add < int >' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/cmdline.h:329:8: note: Technically the member function 'cmdline::parser::add < int >' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/cmdline.h:329:8: note: Technically the member function 'cmdline::parser::add < int >' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/cmdline.h:329:8: performance:inconclusive: Technically the member function 'cmdline::parser::add < long >' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/cmdline.h:329:8: note: Technically the member function 'cmdline::parser::add < long >' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/cmdline.h:329:8: note: Technically the member function 'cmdline::parser::add < long >' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/cmdline.h:589:8: style:inconclusive: Technically the member function 'cmdline::parser::check' can be const. [functionConst] fastp-0.23.4/src/cmdline.h:138:3: style: Class 'cmdline_error' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/cmdline.h:647:6: style: The destructor '~option_without_value' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:625:14: note: Virtual destructor in base class fastp-0.23.4/src/cmdline.h:647:6: note: Destructor in derived class fastp-0.23.4/src/cmdline.h:649:10: style: The function 'has_value' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:627:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:649:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:651:10: style: The function 'set' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:628:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:651:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:656:10: style: The function 'set' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:629:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:656:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:660:10: style: The function 'has_set' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:630:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:660:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:664:10: style: The function 'valid' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:631:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:664:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:668:10: style: The function 'must' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:632:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:668:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:672:24: style: The function 'name' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:634:32: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:672:24: note: Function in derived class fastp-0.23.4/src/cmdline.h:676:10: style: The function 'short_name' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:635:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:676:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:680:24: style: The function 'description' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:636:32: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:680:24: note: Function in derived class fastp-0.23.4/src/cmdline.h:684:17: style: The function 'short_description' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:637:25: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:684:17: note: Function in derived class fastp-0.23.4/src/cmdline.h:707:6: style: The destructor '~option_with_value' overrides a destructor in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:625:14: note: Virtual destructor in base class fastp-0.23.4/src/cmdline.h:707:6: note: Destructor in derived class fastp-0.23.4/src/cmdline.h:713:10: style: The function 'has_value' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:627:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:713:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:715:10: style: The function 'set' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:628:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:715:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:719:10: style: The function 'set' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:629:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:719:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:730:10: style: The function 'has_set' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:630:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:730:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:734:10: style: The function 'valid' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:631:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:734:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:739:10: style: The function 'must' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:632:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:739:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:743:24: style: The function 'name' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:634:32: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:743:24: note: Function in derived class fastp-0.23.4/src/cmdline.h:747:10: style: The function 'short_name' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:635:18: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:747:10: note: Function in derived class fastp-0.23.4/src/cmdline.h:751:24: style: The function 'description' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:636:32: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:751:24: note: Function in derived class fastp-0.23.4/src/cmdline.h:755:17: style: The function 'short_description' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:637:25: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:755:17: note: Function in derived class fastp-0.23.4/src/cmdline.h:792:7: style: The function 'read' overrides a function in a base class but is not marked with a 'override' specifier. [missingOverride] fastp-0.23.4/src/cmdline.h:767:15: note: Virtual function in base class fastp-0.23.4/src/cmdline.h:792:7: note: Function in derived class fastp-0.23.4/src/main.cpp:162:13: style: The if condition is the same as the previous if condition [duplicateCondition] fastp-0.23.4/src/main.cpp:158:13: note: First condition fastp-0.23.4/src/main.cpp:162:13: note: Second condition fastp-0.23.4/src/cmdline.h:333:20: performance: Function parameter 'def' should be passed by const reference. [passedByValue] fastp-0.23.4/src/cmdline.h:444:26: style: Variable 'lookup[initial]' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/cmdline.h:316:28: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/cmdline.h:439:28: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/cmdline.h:520:28: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/nucleotidetree.h:29:10: style:inconclusive: Technically the member function 'NucleotideTree::addSeq' can be const. [functionConst] fastp-0.23.4/src/nucleotidetree.cpp:42:22: note: Technically the member function 'NucleotideTree::addSeq' can be const. fastp-0.23.4/src/nucleotidetree.h:29:10: note: Technically the member function 'NucleotideTree::addSeq' can be const. fastp-0.23.4/src/nucleotidetree.h:30:12: style:inconclusive: Technically the member function 'NucleotideTree::getDominantPath' can be const. [functionConst] fastp-0.23.4/src/nucleotidetree.cpp:57:24: note: Technically the member function 'NucleotideTree::getDominantPath' can be const. fastp-0.23.4/src/nucleotidetree.h:30:12: note: Technically the member function 'NucleotideTree::getDominantPath' can be const. fastp-0.23.4/src/nucleotidetree.cpp:34:5: warning: Class 'NucleotideTree' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/nucleotidetree.cpp:34:5: warning: Class 'NucleotideTree' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/options.cpp:70:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/options.cpp:108:13: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/options.cpp:470:43: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable in [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/options.cpp:9:5: performance: Variable 'in1' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/options.cpp:10:5: performance: Variable 'in2' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/options.cpp:11:5: performance: Variable 'out1' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/options.cpp:12:5: performance: Variable 'out2' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/options.cpp:13:5: performance: Variable 'reportTitle' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/options.h:285:10: performance:inconclusive: Technically the member function 'Options::init' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/options.cpp:33:15: note: Technically the member function 'Options::init' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/options.h:285:10: note: Technically the member function 'Options::init' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/options.h:286:10: style:inconclusive: Technically the member function 'Options::isPaired' can be const. [functionConst] fastp-0.23.4/src/options.cpp:36:15: note: Technically the member function 'Options::isPaired' can be const. fastp-0.23.4/src/options.h:286:10: note: Technically the member function 'Options::isPaired' can be const. fastp-0.23.4/src/options.h:289:10: style:inconclusive: Technically the member function 'Options::polyXTrimmingEnabled' can be const. [functionConst] fastp-0.23.4/src/options.cpp:48:15: note: Technically the member function 'Options::polyXTrimmingEnabled' can be const. fastp-0.23.4/src/options.h:289:10: note: Technically the member function 'Options::polyXTrimmingEnabled' can be const. fastp-0.23.4/src/options.h:290:12: style:inconclusive: Technically the member function 'Options::getAdapter1' can be const. [functionConst] fastp-0.23.4/src/options.cpp:497:17: note: Technically the member function 'Options::getAdapter1' can be const. fastp-0.23.4/src/options.h:290:12: note: Technically the member function 'Options::getAdapter1' can be const. fastp-0.23.4/src/options.h:291:12: style:inconclusive: Technically the member function 'Options::getAdapter2' can be const. [functionConst] fastp-0.23.4/src/options.cpp:504:17: note: Technically the member function 'Options::getAdapter2' can be const. fastp-0.23.4/src/options.h:291:12: note: Technically the member function 'Options::getAdapter2' can be const. fastp-0.23.4/src/options.h:293:20: performance:inconclusive: Technically the member function 'Options::makeListFromFileByLine' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/options.cpp:467:25: note: Technically the member function 'Options::makeListFromFileByLine' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/options.h:293:20: note: Technically the member function 'Options::makeListFromFileByLine' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/options.cpp:446:41: performance: Function parameter 'blacklistFile1' should be passed by const reference. [passedByValue] fastp-0.23.4/src/options.cpp:446:64: performance: Function parameter 'blacklistFile2' should be passed by const reference. [passedByValue] fastp-0.23.4/src/options.cpp:467:55: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/options.cpp:65:56: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/overlapanalysis.cpp:46:38: style: Redundant condition: diff>overlapDiffLimit. 'A || (!A && B)' is equivalent to 'A || B' [redundantCondition] fastp-0.23.4/src/overlapanalysis.cpp:80:38: style: Redundant condition: diff>overlapDiffLimit. 'A || (!A && B)' is equivalent to 'A || B' [redundantCondition] fastp-0.23.4/src/overlapanalysis.cpp:10:64: style:inconclusive: Function 'analyze' argument 3 names different: declaration 'diffLimit' definition 'overlapDiffLimit'. [funcArgNamesDifferent] fastp-0.23.4/src/overlapanalysis.h:29:58: note: Function 'analyze' argument 3 names different: declaration 'diffLimit' definition 'overlapDiffLimit'. fastp-0.23.4/src/overlapanalysis.cpp:10:64: note: Function 'analyze' argument 3 names different: declaration 'diffLimit' definition 'overlapDiffLimit'. fastp-0.23.4/src/overlapanalysis.cpp:15:49: style: Parameter 'r1' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/overlapanalysis.cpp:25:21: style: Variable 'overlap_len' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/overlapanalysis.cpp:27:14: style: Variable 'diff' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/peprocessor.cpp:25:39: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable atomic_long [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/peprocessor.cpp:232:5: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/peprocessor.cpp:360:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/peprocessor.cpp:795:21: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/peprocessor.cpp:801:21: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/peprocessor.cpp:810:21: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/peprocessor.cpp:935:17: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/peprocessor.cpp:943:21: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/peprocessor.cpp:1002:13: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/peprocessor.cpp:15:19: warning: Member variable 'PairEndProcessor::mLeftInputLists' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/peprocessor.cpp:15:19: warning: Member variable 'PairEndProcessor::mRightInputLists' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/peprocessor.cpp:17:5: performance: Variable 'mLeftReaderFinished' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/peprocessor.cpp:18:5: performance: Variable 'mRightReaderFinished' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/peprocessor.h:37:10: style:inconclusive: Technically the member function 'PairEndProcessor::initConfig' can be const. [functionConst] fastp-0.23.4/src/peprocessor.cpp:120:24: note: Technically the member function 'PairEndProcessor::initConfig' can be const. fastp-0.23.4/src/peprocessor.h:37:10: note: Technically the member function 'PairEndProcessor::initConfig' can be const. fastp-0.23.4/src/peprocessor.h:41:9: style:inconclusive: Technically the member function 'PairEndProcessor::getPeakInsertSize' can be const. [functionConst] fastp-0.23.4/src/peprocessor.cpp:334:23: note: Technically the member function 'PairEndProcessor::getPeakInsertSize' can be const. fastp-0.23.4/src/peprocessor.h:41:9: note: Technically the member function 'PairEndProcessor::getPeakInsertSize' can be const. fastp-0.23.4/src/peprocessor.h:42:10: style:inconclusive: Technically the member function 'PairEndProcessor::writerTask' can be const. [functionConst] fastp-0.23.4/src/peprocessor.cpp:1037:24: note: Technically the member function 'PairEndProcessor::writerTask' can be const. fastp-0.23.4/src/peprocessor.h:42:10: note: Technically the member function 'PairEndProcessor::writerTask' can be const. fastp-0.23.4/src/peprocessor.cpp:20:5: warning: Class 'PairEndProcessor' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/peprocessor.cpp:20:5: warning: Class 'PairEndProcessor' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/peprocessor.cpp:380:39: style: Condition 'pcount' is always true [knownConditionTrueFalse] fastp-0.23.4/src/peprocessor.cpp:380:18: note: Assuming that condition 'pcount' is not redundant fastp-0.23.4/src/peprocessor.cpp:359:24: note: Assuming that condition 'leftPack->count!=rightPack->count' is not redundant fastp-0.23.4/src/peprocessor.cpp:380:39: note: Condition 'pcount' is always true fastp-0.23.4/src/peprocessor.cpp:808:25: style: Condition 'mLeftWriter' is always true [knownConditionTrueFalse] fastp-0.23.4/src/peprocessor.cpp:941:25: style: Condition 'mLeftWriter' is always true [knownConditionTrueFalse] fastp-0.23.4/src/peprocessor.cpp:694:74: style: Parameter 'ov' can be declared as reference to const [constParameterReference] fastp-0.23.4/src/peprocessor.cpp:469:28: style: Variable 'isizeEvaluated' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/peprocessor.cpp:720:31: style: Variable 'splitSizeReEvaluated' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/peprocessor.cpp:865:31: style: Variable 'splitSizeReEvaluated' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/polyx.cpp:16:29: style:inconclusive: Function 'trimPolyG' argument 1 names different: declaration 'r1' definition 'r'. [funcArgNamesDifferent] fastp-0.23.4/src/polyx.h:19:33: note: Function 'trimPolyG' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/polyx.cpp:16:29: note: Function 'trimPolyG' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/polyx.cpp:49:29: style:inconclusive: Function 'trimPolyX' argument 1 names different: declaration 'r1' definition 'r'. [funcArgNamesDifferent] fastp-0.23.4/src/polyx.h:21:33: note: Function 'trimPolyX' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/polyx.cpp:49:29: note: Function 'trimPolyX' argument 1 names different: declaration 'r1' definition 'r'. fastp-0.23.4/src/polyx.cpp:108:36: warning: Uninitialized variable: poly [uninitvar] fastp-0.23.4/src/polyx.cpp:103:31: note: Assuming condition is false fastp-0.23.4/src/polyx.cpp:108:36: note: Uninitialized variable: poly fastp-0.23.4/src/read.cpp:42:7: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/read.cpp:49:20: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/read.h:22:12: style:inconclusive: Technically the member function 'Read::firstIndex' can be const. [functionConst] fastp-0.23.4/src/read.cpp:88:14: note: Technically the member function 'Read::firstIndex' can be const. fastp-0.23.4/src/read.h:22:12: note: Technically the member function 'Read::firstIndex' can be const. fastp-0.23.4/src/read.h:23:12: style:inconclusive: Technically the member function 'Read::lastIndex' can be const. [functionConst] fastp-0.23.4/src/read.cpp:76:14: note: Technically the member function 'Read::lastIndex' can be const. fastp-0.23.4/src/read.h:23:12: note: Technically the member function 'Read::lastIndex' can be const. fastp-0.23.4/src/read.h:25:9: style:inconclusive: Technically the member function 'Read::lowQualCount' can be const. [functionConst] fastp-0.23.4/src/read.cpp:103:11: note: Technically the member function 'Read::lowQualCount' can be const. fastp-0.23.4/src/read.h:25:9: note: Technically the member function 'Read::lowQualCount' can be const. fastp-0.23.4/src/read.h:26:9: style:inconclusive: Technically the member function 'Read::length' can be const. [functionConst] fastp-0.23.4/src/read.cpp:112:11: note: Technically the member function 'Read::length' can be const. fastp-0.23.4/src/read.h:26:9: note: Technically the member function 'Read::length' can be const. fastp-0.23.4/src/read.h:27:12: style:inconclusive: Technically the member function 'Read::toString' can be const. [functionConst] fastp-0.23.4/src/read.cpp:116:14: note: Technically the member function 'Read::toString' can be const. fastp-0.23.4/src/read.h:27:12: note: Technically the member function 'Read::toString' can be const. fastp-0.23.4/src/read.h:28:12: style:inconclusive: Technically the member function 'Read::toStringWithTag' can be const. [functionConst] fastp-0.23.4/src/read.cpp:176:14: note: Technically the member function 'Read::toStringWithTag' can be const. fastp-0.23.4/src/read.h:28:12: note: Technically the member function 'Read::toStringWithTag' can be const. fastp-0.23.4/src/read.cpp:16:2: warning: Class 'Read' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/read.cpp:16:2: warning: Class 'Read' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/read.cpp:60:2: error: Memory leak: strand [memleak] fastp-0.23.4/src/read.cpp:232:6: style: The scope of the variable 'lowQualDiff' can be reduced. [variableScope] fastp-0.23.4/src/read.cpp:146:57: performance: Function parameter 'tag' should be passed by const reference. [passedByValue] fastp-0.23.4/src/read.cpp:176:37: performance: Function parameter 'tag' should be passed by const reference. [passedByValue] fastp-0.23.4/src/read.cpp:59:10: style: Variable 'strand' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/read.cpp:301:8: style: Variable 'merged' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/read.cpp:59:16: style: Variable 'strand' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/read.cpp:232:18: style: Variable 'lowQualDiff' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/singleproducersingleconsumerlist.h:53:12: warning: Member variable 'LockFreeListItem < Read * >::value' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/readpool.h:23:10: performance:inconclusive: Technically the member function 'ReadPool::cleanup' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/readpool.cpp:33:16: note: Technically the member function 'ReadPool::cleanup' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/readpool.h:23:10: note: Technically the member function 'ReadPool::cleanup' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/singleproducersingleconsumerlist.h:87:19: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < Read * >::size' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:90:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < Read * >::canBeConsumed' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:116:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < Read * >::isProducerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:119:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < Read * >::isConsumerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList < Read * >' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList < Read * >' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/singleproducersingleconsumerlist.h:48:12: style: Struct 'LockFreeListItem < Read * >' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/readpool.cpp:39:5: warning: Possible leak in public function. The pointer 'mBufferLists' is not deallocated before it is allocated. [publicAllocationError] fastp-0.23.4/src/readpool.cpp:22:37: style:inconclusive: Function 'input' argument 2 names different: declaration 'r' definition 'data'. [funcArgNamesDifferent] fastp-0.23.4/src/readpool.h:20:31: note: Function 'input' argument 2 names different: declaration 'r' definition 'data'. fastp-0.23.4/src/readpool.cpp:22:37: note: Function 'input' argument 2 names different: declaration 'r' definition 'data'. fastp-0.23.4/src/seprocessor.cpp:136:5: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/seprocessor.cpp:284:54: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable stdout [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/seprocessor.cpp:374:17: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/seprocessor.cpp:382:21: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/seprocessor.cpp:439:13: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/seprocessor.cpp:14:21: warning: Member variable 'SingleEndProcessor::mInputLists' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/seprocessor.cpp:16:5: performance: Variable 'mReaderFinished' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/seprocessor.h:35:10: style:inconclusive: Technically the member function 'SingleEndProcessor::initConfig' can be const. [functionConst] fastp-0.23.4/src/seprocessor.cpp:65:26: note: Technically the member function 'SingleEndProcessor::initConfig' can be const. fastp-0.23.4/src/seprocessor.h:35:10: note: Technically the member function 'SingleEndProcessor::initConfig' can be const. fastp-0.23.4/src/seprocessor.h:38:10: style:inconclusive: Technically the member function 'SingleEndProcessor::writerTask' can be const. [functionConst] fastp-0.23.4/src/seprocessor.cpp:458:26: note: Technically the member function 'SingleEndProcessor::writerTask' can be const. fastp-0.23.4/src/seprocessor.h:38:10: note: Technically the member function 'SingleEndProcessor::writerTask' can be const. fastp-0.23.4/src/seprocessor.cpp:18:5: warning: Class 'SingleEndProcessor' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/seprocessor.cpp:18:5: warning: Class 'SingleEndProcessor' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/seprocessor.cpp:326:31: style: Variable 'splitSizeReEvaluated' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/sequence.cpp:16:10: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/sequence.cpp:89:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/sequence.cpp:3:11: warning:inconclusive: Member variable 'Sequence::mStr' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/sequence.h:16:10: style:inconclusive: Technically the member function 'Sequence::print' can be const. [functionConst] fastp-0.23.4/src/sequence.cpp:15:16: note: Technically the member function 'Sequence::print' can be const. fastp-0.23.4/src/sequence.h:16:10: note: Technically the member function 'Sequence::print' can be const. fastp-0.23.4/src/sequence.h:17:9: style:inconclusive: Technically the member function 'Sequence::length' can be const. [functionConst] fastp-0.23.4/src/sequence.cpp:19:15: note: Technically the member function 'Sequence::length' can be const. fastp-0.23.4/src/sequence.h:17:9: note: Technically the member function 'Sequence::length' can be const. fastp-0.23.4/src/stats.cpp:380:5: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.cpp:387:19: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.cpp:562:176: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.cpp:690:9: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable hex [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.cpp:735:53: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.cpp:810:53: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable endl [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.cpp:929:17: debug: valueFlowConditionExpressions bailout: Skipping function due to incomplete variable cerr [valueFlowBailoutIncompleteVar] fastp-0.23.4/src/stats.h:39:10: style:inconclusive: Technically the member function 'Stats::isLongRead' can be const. [functionConst] fastp-0.23.4/src/stats.cpp:631:13: note: Technically the member function 'Stats::isLongRead' can be const. fastp-0.23.4/src/stats.h:39:10: note: Technically the member function 'Stats::isLongRead' can be const. fastp-0.23.4/src/stats.h:41:9: style:inconclusive: Technically the member function 'Stats::getMeanLength' can be const. [functionConst] fastp-0.23.4/src/stats.cpp:218:12: note: Technically the member function 'Stats::getMeanLength' can be const. fastp-0.23.4/src/stats.h:41:9: note: Technically the member function 'Stats::getMeanLength' can be const. fastp-0.23.4/src/stats.h:52:12: performance:inconclusive: Technically the member function 'Stats::kmer3' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/stats.cpp:702:15: note: Technically the member function 'Stats::kmer3' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/stats.h:52:12: note: Technically the member function 'Stats::kmer3' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/stats.h:53:12: performance:inconclusive: Technically the member function 'Stats::kmer2' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/stats.cpp:711:15: note: Technically the member function 'Stats::kmer2' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/stats.h:53:12: note: Technically the member function 'Stats::kmer2' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/stats.h:55:10: style:inconclusive: Technically the member function 'Stats::overRepPassed' can be const. [functionConst] fastp-0.23.4/src/stats.cpp:530:13: note: Technically the member function 'Stats::overRepPassed' can be const. fastp-0.23.4/src/stats.h:55:10: note: Technically the member function 'Stats::overRepPassed' can be const. fastp-0.23.4/src/stats.cpp:50:5: warning: Class 'Stats' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/stats.cpp:50:5: warning: Class 'Stats' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/stats.cpp:77:16: warning: Possible null pointer dereference: newBuf [nullPointer] fastp-0.23.4/src/stats.cpp:70:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:74:27: note: mCycleQ30Bases[i] is assigned 'newBuf' here. fastp-0.23.4/src/stats.cpp:76:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:77:16: note: Null pointer dereference fastp-0.23.4/src/stats.cpp:78:16: warning: Possible null pointer dereference: newBuf [nullPointer] fastp-0.23.4/src/stats.cpp:70:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:74:27: note: mCycleQ30Bases[i] is assigned 'newBuf' here. fastp-0.23.4/src/stats.cpp:76:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:78:16: note: Null pointer dereference fastp-0.23.4/src/stats.cpp:83:16: warning: Possible null pointer dereference: newBuf [nullPointer] fastp-0.23.4/src/stats.cpp:70:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:74:27: note: mCycleQ30Bases[i] is assigned 'newBuf' here. fastp-0.23.4/src/stats.cpp:76:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:76:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:82:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:83:16: note: Null pointer dereference fastp-0.23.4/src/stats.cpp:84:16: warning: Possible null pointer dereference: newBuf [nullPointer] fastp-0.23.4/src/stats.cpp:70:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:74:27: note: mCycleQ30Bases[i] is assigned 'newBuf' here. fastp-0.23.4/src/stats.cpp:76:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:76:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:82:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:84:16: note: Null pointer dereference fastp-0.23.4/src/stats.cpp:89:16: warning: Possible null pointer dereference: newBuf [nullPointer] fastp-0.23.4/src/stats.cpp:70:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:74:27: note: mCycleQ30Bases[i] is assigned 'newBuf' here. fastp-0.23.4/src/stats.cpp:76:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:76:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:82:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:82:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:88:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:89:16: note: Null pointer dereference fastp-0.23.4/src/stats.cpp:90:16: warning: Possible null pointer dereference: newBuf [nullPointer] fastp-0.23.4/src/stats.cpp:70:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:74:27: note: mCycleQ30Bases[i] is assigned 'newBuf' here. fastp-0.23.4/src/stats.cpp:76:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:76:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:82:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:82:16: note: newBuf is assigned 'new long[newBufLen]' here. fastp-0.23.4/src/stats.cpp:88:18: note: Assignment 'newBuf=new long[newBufLen]', assigned value is symbolic=mCycleQ30Bases[i] fastp-0.23.4/src/stats.cpp:90:16: note: Null pointer dereference fastp-0.23.4/src/stats.cpp:848:18: style: Local variable 'b' shadows outer variable [shadowVariable] fastp-0.23.4/src/stats.cpp:844:14: note: Shadowed declaration fastp-0.23.4/src/stats.cpp:848:18: note: Shadow variable fastp-0.23.4/src/stats.cpp:386:46: performance: Function parameter 'padding' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:521:46: performance: Function parameter 'filteringType' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:521:68: performance: Function parameter 'readName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:546:49: performance: Function parameter 'filteringType' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:546:71: performance: Function parameter 'readName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:635:50: performance: Function parameter 'filteringType' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:635:72: performance: Function parameter 'readName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:719:53: performance: Function parameter 'filteringType' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:719:75: performance: Function parameter 'readName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:794:54: performance: Function parameter 'filteringType' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:794:76: performance: Function parameter 'readName' should be passed by const reference. [passedByValue] fastp-0.23.4/src/stats.cpp:530:35: style: Parameter 'seq' can be declared as reference to const [constParameterReference] fastp-0.23.4/src/stats.cpp:176:10: style: Variable 'alphabets' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:396:12: style: Variable 'qualNames' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:400:17: style: Variable 'curve' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/stats.cpp:417:12: style: Variable 'contentNames' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:421:17: style: Variable 'curve' can be declared as pointer to const [constVariablePointer] fastp-0.23.4/src/stats.cpp:476:35: style: Parameter 'list' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/stats.cpp:486:35: style: Parameter 'list' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/stats.cpp:486:57: style: Parameter 'coords' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/stats.cpp:511:33: style: Parameter 'list' can be declared as pointer to const [constParameterPointer] fastp-0.23.4/src/stats.cpp:733:12: style: Variable 'alphabets' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:734:12: style: Variable 'colors' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:808:12: style: Variable 'alphabets' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:809:12: style: Variable 'colors' can be declared as const array [constVariable] fastp-0.23.4/src/stats.cpp:855:26: performance: Ineffective call of function 'substr' because a prefix of the string is assigned to itself. Use resize() or pop_back() instead. [uselessCallsSubstr] fastp-0.23.4/src/stats.cpp:550:9: style: Variable 'displayed' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/stats.cpp:556:12: style: Variable 'title' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/stats.cpp:641:18: style: Variable 'title' is assigned a value that is never used. [unreadVariable] fastp-0.23.4/src/stats.cpp:129:70: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:132:70: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:460:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:564:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:587:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:925:74: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:949:60: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/stats.cpp:960:62: performance: Prefer prefix ++/-- operators for non-primitive types. [postfixOperator] fastp-0.23.4/src/threadconfig.h:39:10: style:inconclusive: Technically the member function 'ThreadConfig::canBeStopped' can be const. [functionConst] fastp-0.23.4/src/threadconfig.cpp:139:20: note: Technically the member function 'ThreadConfig::canBeStopped' can be const. fastp-0.23.4/src/threadconfig.h:39:10: note: Technically the member function 'ThreadConfig::canBeStopped' can be const. fastp-0.23.4/src/threadconfig.cpp:9:5: warning: Class 'ThreadConfig' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/threadconfig.cpp:9:5: warning: Class 'ThreadConfig' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/threadconfig.cpp:67:38: performance: Function parameter 'filename1' should be passed by const reference. [passedByValue] fastp-0.23.4/src/threadconfig.cpp:72:38: performance: Function parameter 'filename1' should be passed by const reference. [passedByValue] fastp-0.23.4/src/threadconfig.cpp:72:56: performance: Function parameter 'filename2' should be passed by const reference. [passedByValue] fastp-0.23.4/src/umiprocessor.h:17:10: style:inconclusive: Technically the member function 'UmiProcessor::addUmiToName' can be const. [functionConst] fastp-0.23.4/src/umiprocessor.cpp:63:20: note: Technically the member function 'UmiProcessor::addUmiToName' can be const. fastp-0.23.4/src/umiprocessor.h:17:10: note: Technically the member function 'UmiProcessor::addUmiToName' can be const. fastp-0.23.4/src/umiprocessor.cpp:63:49: performance: Function parameter 'umi' should be passed by const reference. [passedByValue] fastp-0.23.4/src/unittest.h:14:10: performance:inconclusive: Technically the member function 'UnitTest::report' can be static (but you may consider moving to unnamed namespace). [functionStatic] fastp-0.23.4/src/unittest.cpp:34:16: note: Technically the member function 'UnitTest::report' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/unittest.h:14:10: note: Technically the member function 'UnitTest::report' can be static (but you may consider moving to unnamed namespace). fastp-0.23.4/src/unittest.cpp:34:43: performance: Function parameter 'message' should be passed by const reference. [passedByValue] fastp-0.23.4/src/writer.cpp:31:2: performance: Variable 'mFilename' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/writer.h:43:7: style:inconclusive: Technically the member function 'Writer::isZipped' can be const. [functionConst] fastp-0.23.4/src/writer.cpp:147:14: note: Technically the member function 'Writer::isZipped' can be const. fastp-0.23.4/src/writer.h:43:7: note: Technically the member function 'Writer::isZipped' can be const. fastp-0.23.4/src/writer.h:48:9: style:inconclusive: Technically the member function 'Writer::filename' can be const. [functionConst] fastp-0.23.4/src/writer.cpp:56:16: note: Technically the member function 'Writer::filename' can be const. fastp-0.23.4/src/writer.h:48:9: note: Technically the member function 'Writer::filename' can be const. fastp-0.23.4/src/writer.h:48:9: performance: Function 'filename()' should return member 'mFilename' by const reference. [returnByReference] fastp-0.23.4/src/writer.cpp:61:12: style: C-style pointer casting [cstyleCast] fastp-0.23.4/src/writer.cpp:29:37: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/writer.cpp:106:9: style: Unused variable: written [unusedVariable] fastp-0.23.4/src/writerthread.cpp:40:9: style: Obsolescent function 'usleep' called. It is recommended to use 'nanosleep' or 'setitimer' instead. [prohibitedusleepCalled] fastp-0.23.4/src/singleproducersingleconsumerlist.h:53:12: warning: Member variable 'LockFreeListItem < string * >::value' is not initialized in the constructor. [uninitMemberVar] fastp-0.23.4/src/writerthread.cpp:12:5: performance: Variable 'mFilename' is assigned in constructor body. Consider performing initialization in initialization list. [useInitializationList] fastp-0.23.4/src/writerthread.h:26:10: style:inconclusive: Technically the member function 'WriterThread::isCompleted' can be const. [functionConst] fastp-0.23.4/src/writerthread.cpp:24:20: note: Technically the member function 'WriterThread::isCompleted' can be const. fastp-0.23.4/src/writerthread.h:26:10: note: Technically the member function 'WriterThread::isCompleted' can be const. fastp-0.23.4/src/singleproducersingleconsumerlist.h:87:19: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < string * >::size' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:90:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < string * >::canBeConsumed' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:116:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < string * >::isProducerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:119:17: style:inconclusive: Technically the member function 'SingleProducerSingleConsumerList < string * >::isConsumerFinished' can be const. [functionConst] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList < string * >' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s). [noCopyConstructor] fastp-0.23.4/src/singleproducersingleconsumerlist.h:77:9: warning: Class 'SingleProducerSingleConsumerList < string * >' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s). [noOperatorEq] fastp-0.23.4/src/singleproducersingleconsumerlist.h:48:12: style: Struct 'LockFreeListItem < string * >' has a constructor with 1 argument that is not explicit. [noExplicitConstructor] fastp-0.23.4/src/writerthread.cpp:73:5: warning: Possible leak in public function. The pointer 'mBufferLists' is not deallocated before it is allocated. [publicAllocationError] fastp-0.23.4/src/writerthread.cpp:6:49: performance: Function parameter 'filename' should be passed by const reference. [passedByValue] fastp-0.23.4/src/writerthread.cpp:67:38: performance: Function parameter 'filename1' should be passed by const reference. [passedByValue] fastp-0.23.4/src/htmlreporter.cpp:56:28: warning: Null pointer dereference: preStats1 [ctunullpointer] fastp-0.23.4/src/htmlreporter.cpp:337:8: note: Assuming that condition 'preStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:17: note: Calling function printSummary, 3rd argument is null fastp-0.23.4/src/htmlreporter.cpp:56:28: note: Dereferencing argument preStats1 that is null fastp-0.23.4/src/htmlreporter.cpp:76:29: warning: Null pointer dereference: postStats1 [ctunullpointer] fastp-0.23.4/src/htmlreporter.cpp:352:8: note: Assuming that condition 'postStats1' is not redundant fastp-0.23.4/src/htmlreporter.cpp:331:17: note: Calling function printSummary, 4th argument is null fastp-0.23.4/src/htmlreporter.cpp:76:29: note: Dereferencing argument postStats1 that is null diff: DONE