Index: cppcheck.err =================================================================== diff -u -ra89b5249df072be403141463d7d88728b66a848e -r93223f7d403a66dcb0d72a31802181c1b38e7ce3 --- cppcheck.err (.../cppcheck.err) (revision a89b5249df072be403141463d7d88728b66a848e) +++ cppcheck.err (.../cppcheck.err) (revision 93223f7d403a66dcb0d72a31802181c1b38e7ce3) @@ -1,3 +1,9 @@ +11/03/2022 14:42 +Behrouz NematiPour +[sources/storage/FileHandler.h:95] -> [sources/storage/FileHandler.h:102]: (style) Condition '!createFolder' is always false +[sources/ApplicationPost.cpp:109] -> [sources/ApplicationPost.cpp:111]: (style) Condition '!ok' is always false +[sources/ApplicationPost.cpp:204] -> [sources/ApplicationPost.cpp:206]: (style) Condition '!ok' is always false +[sources/ApplicationPost.cpp:218] -> [sources/ApplicationPost.cpp:220]: (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) { @@ -37,44 +43,83 @@ } } When you see this message it is always safe to reduce the variable scope 1 level. +[sources/cloudsync/CloudSyncController.cpp:500] -> [sources/cloudsync/CloudSyncController.cpp:508]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:662] -> [sources/cloudsync/CloudSyncController.cpp:663]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:673] -> [sources/cloudsync/CloudSyncController.cpp:675]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:701] -> [sources/cloudsync/CloudSyncController.cpp:702]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:812] -> [sources/cloudsync/CloudSyncController.cpp:813]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/cloudsync/CloudSyncController.cpp:901] -> [sources/cloudsync/CloudSyncController.cpp:902]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/device/DeviceController.cpp:255] -> [sources/device/DeviceController.cpp:262]: (style) Variable 'mCIsReady' is reassigned a value before the old one has been used. +[sources/device/DeviceController.cpp:304] -> [sources/device/DeviceController.cpp:311]: (style) Variable 'mCIsReady' is reassigned a value before the old one has been used. +[sources/device/DeviceController.cpp:311] -> [sources/device/DeviceController.cpp:313]: (style) Variable 'mCIsReady' 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/AlarmGenerator.cpp:68]: (style) The function 'ALARM_ID_BLE_CUFF' is never used. -[sources/view/VTreatmentCreate.cpp:124]: (style) The function 'doCancel' is never used. -[sources/view/settings/VNetworkModel.cpp:223]: (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:114]: (style) The function 'doConfirm' is never used. -[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:73]: (style) The function 'doDisinfectChemical' is never used. -[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:63]: (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/model/settings/MSettings.h:56]: (style) The KeyValue::operator= does not conform to standard C/C++ behaviour. To conform to standard C/C++ behaviour, return a reference to self (such as: 'KeyValue &KeyValue::operator=(..) { .. return *this; }'. For safety reasons it might be better to not fix this message. If you think that safety is always more important than conformance then please ignore/suppress this message. For more details about this topic, see the book "Effective C++" by Scott Meyers. +[sources/model/settings/MSettings.h:44]: (warning) The class 'KeyValue' has 'operator=' but lack of 'copy constructor'. +[sources/storage/Logger.cpp:259] -> [sources/storage/Logger.cpp:261]: (style) Variable 'ok' is reassigned a value before the old one has been used. +[sources/storage/Logger.cpp:189]: (style) The scope of the variable 'notified' 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/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:95]: (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:88]: (style) The function 'doDisinfectCancel' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:127]: (style) The function 'doDisinfectChemical' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:99]: (style) The function 'doDisinfectClear' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:132]: (style) The function 'doDisinfectConfirm' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:109]: (style) The function 'doDisinfectFlush' is never used. +[sources/view/hd/adjustment/disinfect/VDisinfectAdjustDisinfect.cpp:118]: (style) The function 'doDisinfectHeat' is never used. +[sources/gui/GuiView.cpp:250]: (style) The function 'doExportListInsert' is never used. +[sources/gui/GuiView.cpp:293]: (style) The function 'doExportListPercent' is never used. +[sources/gui/GuiView.cpp:281]: (style) The function 'doExportListSelect' 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:44]: (style) The function 'doInit' is never used. -[sources/view/VEventSpy.cpp:139]: (style) The function 'doMouseReset' is never used. +[sources/view/VEventSpy.cpp:149]: (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/VTreatmentCreate (legacy).cpp:111]: (style) The function 'doResetCreateTreatment' is never used. +[sources/view/hd/alarm/VAlarmActiveList.cpp:106]: (style) The function 'doRequest' is never used. +[sources/view/hd/adjustment/common/VCommonAdjustmentVitals.cpp:99]: (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:347]: (style) The function 'doSetDNS' is never used. -[sources/view/settings/VNetworkModel.cpp:329]: (style) The function 'doSetGateway' is never used. -[sources/view/settings/VNetworkModel.cpp:320]: (style) The function 'doSetIPAddress' is never used. -[sources/view/settings/VNetworkModel.cpp:338]: (style) The function 'doSetSubnetMask' is never used. -[sources/view/hd/alarm/VAlarmStatus.cpp:76]: (style) The function 'doSilence' 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:78]: (style) The function 'doSilence' is never used. +[sources/view/hd/adjustment/common/VCommonAdjustmentVitals.cpp:87]: (style) The function 'doSkip' is never used. [sources/view/hd/adjustment/treatment/VTreatmentAdjustmentSaline.cpp:58]: (style) The function 'doStop' is never used. -[sources/view/VEventSpy.cpp:182]: (style) The function 'doTouchReset' is never used. +[sources/view/VEventSpy.cpp:192]: (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:94]: (style) The function 'doValidation' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:57]: (style) The function 'doUserActionEnd' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:67]: (style) The function 'doUserActionOk' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:37]: (style) The function 'doUserActionResume' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:47]: (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/storage/FileHandler.cpp:166]: (style) The function 'removeFiles' is never used. -[sources/view/settings/VNetworkModel.cpp:166]: (style) The function 'roleNames' is never used. -[sources/view/VAdjustmentResponseBase.cpp:28]: (style) The function 'text' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1159]: (style) The function 'mimic' is never used. +[sources/storage/FileHandler.cpp:162]: (style) The function 'moveFolder' is never used. +[sources/storage/FileHandler.cpp:203]: (style) The function 'removeFiles' is never used. +[sources/storage/FileHandler.cpp:183]: (style) The function 'removeFolder' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1239]: (style) The function 'requestBattery' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1226]: (style) The function 'requestInformation' is never used. +[sources/bluetooth/BluetoothInterface.cpp:1207]: (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.