Index: cppcheck.err =================================================================== diff -u -r4f8cbe0045e2a06a67a44894bb02b41a02eca5dd -r1ce869e6f7dcbeee8a7ff945e436e7c3ac07331e --- cppcheck.err (.../cppcheck.err) (revision 4f8cbe0045e2a06a67a44894bb02b41a02eca5dd) +++ cppcheck.err (.../cppcheck.err) (revision 1ce869e6f7dcbeee8a7ff945e436e7c3ac07331e) @@ -1 +1,97 @@ -./cppcheck.sh: line 17: cppcheck: command not found +[sources/ApplicationPost.cpp:104] -> [sources/ApplicationPost.cpp:106]: (style) Condition '!ok' is always false +[sources/ApplicationPost.cpp:183] -> [sources/ApplicationPost.cpp:185]: (style) Condition '!ok' is always false +[sources/ApplicationPost.cpp:197] -> [sources/ApplicationPost.cpp:199]: (style) Condition '!ok' is always false +[sources/MainTimer.cpp:69]: (style) The scope of the variable 'oH' can be reduced. Warning: Be careful when fixing this message, especially when there are inner loops. Here is an example where cppcheck will write that the scope for 'i' can be reduced: +void f(int x) +{ + int i = 0; + if (x) { + // it's safe to move 'int i = 0;' here + for (int n = 0; n < 10; ++n) { + // it is possible but not safe to move 'int i = 0;' here + do_something(&i); + } + } +} +When you see this message it is always safe to reduce the variable scope 1 level. +[sources/MainTimer.cpp:69]: (style) The scope of the variable 'oM' can be reduced. Warning: Be careful when fixing this message, especially when there are inner loops. Here is an example where cppcheck will write that the scope for 'i' can be reduced: +void f(int x) +{ + int i = 0; + if (x) { + // it's safe to move 'int i = 0;' here + for (int n = 0; n < 10; ++n) { + // it is possible but not safe to move 'int i = 0;' here + do_something(&i); + } + } +} +When you see this message it is always safe to reduce the variable scope 1 level. +[sources/MainTimer.cpp:69]: (style) The scope of the variable 'oS' can be reduced. Warning: Be careful when fixing this message, especially when there are inner loops. Here is an example where cppcheck will write that the scope for 'i' can be reduced: +void f(int x) +{ + int i = 0; + if (x) { + // it's safe to move 'int i = 0;' here + for (int n = 0; n < 10; ++n) { + // it is possible but not safe to move 'int i = 0;' here + do_something(&i); + } + } +} +When you see this message it is always safe to reduce the variable scope 1 level. +[sources/cloudsync/CloudSyncController.cpp:472] -> [sources/cloudsync/CloudSyncController.cpp:480]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:619] -> [sources/cloudsync/CloudSyncController.cpp:620]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:646] -> [sources/cloudsync/CloudSyncController.cpp:647]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:712] -> [sources/cloudsync/CloudSyncController.cpp:713]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:801] -> [sources/cloudsync/CloudSyncController.cpp:802]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/model/hd/adjustment/treatment/MTreatmentAdjustUltrafiltrationEditResponse.h:67] -> [sources/model/hd/adjustment/treatment/MTreatmentAdjustUltrafiltrationConfirmResponse.h:61]: (warning) The class 'MAdjustUltrafiltrationEditResponse' defines member variable with name 'stringPrefix' also defined in its parent class 'MAdjustUltrafiltrationConfirmResponse'. +[sources/model/hd/adjustment/treatment/MTreatmentAdjustUltrafiltrationEditResponse.h:79] -> [sources/model/hd/adjustment/treatment/MTreatmentAdjustUltrafiltrationConfirmResponse.h:79]: (warning) The class 'MAdjustUltrafiltrationEditResponse' defines member variable with name '_data' also defined in its parent class 'MAdjustUltrafiltrationConfirmResponse'. +[sources/storage/Settings.h:44]: (style) Unused private function: 'Settings::makeSetting' +[sources/view/VEventSpy.cpp]: (information) The configuration 'DEBUG_BCUFF_MIMIC' was not checked because its code equals another one. +[sources/view/VTreatmentCreate.cpp:132]: (style) The function 'doCancel' is never used. +[sources/view/settings/VNetworkModel.cpp:238]: (style) The function 'doCheckIfConnected' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:93]: (style) The function 'doClearCondition' is never used. +[sources/view/VTreatmentCreate.cpp:121]: (style) The function 'doConfirm' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:74]: (style) The function 'doDisinfectChemical' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:64]: (style) The function 'doDisinfectHeat' is never used. +[sources/view/VTreatmentCreate(legacy).cpp:520]: (style) The function 'doGetOperatingParameterNames' is never used. +[sources/view/VTreatmentCreate(legacy).cpp:539]: (style) The function 'doGetOperatingParameterValues' is never used. +[sources/view/VTreatmentCreate(legacy).cpp:487]: (style) The function 'doGetPrescriptionParameterNames' is never used. +[sources/view/VTreatmentCreate(legacy).cpp:504]: (style) The function 'doGetPrescriptionParameterValues' is never used. +[sources/view/settings/VDateTime.cpp:43]: (style) The function 'doInit' is never used. +[sources/view/VEventSpy.cpp:147]: (style) The function 'doMouseReset' is never used. +[sources/view/hd/adjustment/treatment/VTreatmentAdjustmentUltrafiltrationConfirm.cpp:66]: (style) The function 'doOptionDuration' is never used. +[sources/view/hd/adjustment/treatment/VTreatmentAdjustmentRecirculate.cpp:51]: (style) The function 'doReconnect' is never used. +[sources/view/hd/alarm/VAlarmActiveList.cpp:106]: (style) The function 'doRequest' is never used. +[sources/view/hd/adjustment/common/VCommonAdjustmentVitals.cpp:113]: (style) The function 'doReset' is never used. +[sources/view/VTreatmentCreate(legacy).cpp:111]: (style) The function 'doResetCreateTreatment' is never used. +[sources/view/hd/adjustment/pretreatment/VPreTreatmentAdjustmentWaterSample.cpp:64]: (style) The function 'doResult' is never used. +[sources/view/hd/adjustment/treatment/VTreatmentAdjustmentHeparin.cpp:51]: (style) The function 'doResume' is never used. +[sources/view/settings/VNetworkModel.cpp:362]: (style) The function 'doSetDNS' is never used. +[sources/view/settings/VNetworkModel.cpp:344]: (style) The function 'doSetGateway' is never used. +[sources/view/settings/VNetworkModel.cpp:335]: (style) The function 'doSetIPAddress' is never used. +[sources/view/settings/VNetworkModel.cpp:353]: (style) The function 'doSetSubnetMask' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:76]: (style) The function 'doSilence' is never used. +[sources/view/hd/adjustment/common/VCommonAdjustmentVitals.cpp:93]: (style) The function 'doSkip' is never used. +[sources/view/hd/adjustment/treatment/VTreatmentAdjustmentSaline.cpp:58]: (style) The function 'doStop' is never used. +[sources/view/hd/adjustment/common/VCommonAdjustmentVitals.cpp:102]: (style) The function 'doTimeout' is never used. +[sources/view/VEventSpy.cpp:190]: (style) The function 'doTouchReset' is never used. +[sources/view/hd/adjustment/treatment/VTreatmentAdjustmentRecirculate.cpp:75]: (style) The function 'doTreatmentEnd' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:55]: (style) The function 'doUserActionEnd' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:65]: (style) The function 'doUserActionOk' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:35]: (style) The function 'doUserActionResume' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:45]: (style) The function 'doUserActionRinseback' is never used. +[sources/view/VTreatmentCreate.cpp:101]: (style) The function 'doValidation' is never used. +[sources/utility/types.cpp:28]: (style) The function 'floatCompare' is never used. +[sources/utility/types.cpp:44]: (style) The function 'getBits' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1154]: (style) The function 'mimic' is never used. +[sources/storage/FileHandler.cpp:165]: (style) The function 'moveFolder' is never used. +[sources/storage/FileHandler.cpp:206]: (style) The function 'removeFiles' is never used. +[sources/storage/FileHandler.cpp:186]: (style) The function 'removeFolder' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1234]: (style) The function 'requestBattery' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1221]: (style) The function 'requestInformation' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1202]: (style) The function 'requestMeasurements' is never used. +[sources/view/settings/VBluetooth.cpp:79]: (style) The function 'roleNames' is never used. +[sources/MainTimer.cpp:116]: (style) The function 'timerEvent' is never used. +(information) Cppcheck cannot find all the include files. Cppcheck can check the code without the include files found. But the results will probably be more accurate if all the include files are found. Please check your project's include directories and add all of them as include directories for Cppcheck. To see what files Cppcheck cannot find use --check-config.