Index: etc/reports/cppcheck/cppcheck.err =================================================================== diff -u -r6ac0e78ba8a65d86f67ce866ed879430fe2b2703 -r322c96960f4828abdd6cf9914d2a0797c2887e12 --- etc/reports/cppcheck/cppcheck.err (.../cppcheck.err) (revision 6ac0e78ba8a65d86f67ce866ed879430fe2b2703) +++ etc/reports/cppcheck/cppcheck.err (.../cppcheck.err) (revision 322c96960f4828abdd6cf9914d2a0797c2887e12) @@ -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]