Index: etc/reports/cppcheck/cppcheck.err =================================================================== diff -u -r9bc48695cdb977a80d3279cacdc101d5164f1b4b -r11560b63360025c294edabc6f5f6da97ad1f20c3 --- etc/reports/cppcheck/cppcheck.err (.../cppcheck.err) (revision 9bc48695cdb977a80d3279cacdc101d5164f1b4b) +++ etc/reports/cppcheck/cppcheck.err (.../cppcheck.err) (revision 11560b63360025c294edabc6f5f6da97ad1f20c3) @@ -1,4 +1,4 @@ -04/21/2025 17:38 +04/22/2025 21:26 Behrouz NematiPour /home/denali/Public/luis/application/sources/canbus/MessageAcknowModel.h:58:9: performance: When an object of a class is created, the constructors of all member variables are called consecutively in the order the variables are declared, even if you don't explicitly write them to the initialization list. You could avoid assigning '_sequence' a value by passing the value to the constructor in the initialization list. [useInitializationList] _sequence = vSequence ; @@ -93,7 +93,7 @@ When you see this message it is always safe to reduce the variable scope 1 level. [variableScope] static int oH,oM,oS; int cH,cM,cS; ^ -/home/denali/Public/luis/application/sources/canbus/MessageInterpreter.cpp:557:9: style: Statements following return, break, continue, goto or throw will never be executed. [unreachableCode] +/home/denali/Public/luis/application/sources/canbus/MessageInterpreter.cpp:579:9: style: Statements following return, break, continue, goto or throw will never be executed. [unreachableCode] LOG_APPED(QString("DG,CheckIn," + QVariant(vData).toStringList().join(','))); break; ^ /home/denali/Public/luis/application/sources/cloudsync/CloudSyncController.cpp:310:10: style: The if condition is the same as the previous if condition [duplicateCondition]