Index: cppcheck.err =================================================================== diff -u -r287c4abba7abd146608f79374f440bd45869c4f9 -r76bd975e94c626e498260122375f9ca1c5eb7b3b --- cppcheck.err (.../cppcheck.err) (revision 287c4abba7abd146608f79374f440bd45869c4f9) +++ cppcheck.err (.../cppcheck.err) (revision 76bd975e94c626e498260122375f9ca1c5eb7b3b) @@ -1,4 +1,4 @@ -[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: +[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; @@ -11,7 +11,7 @@ } } 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: +[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; @@ -24,7 +24,7 @@ } } 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: +[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; @@ -37,19 +37,19 @@ } } When you see this message it is always safe to reduce the variable scope 1 level. -[sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationEditResponse.h:35] -> [sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationConfirmResponse.h:33]: (warning) The class 'MAdjustUltrafiltrationEditResponse' defines member variable with name 'stringPrefix' also defined in its parent class 'MAdjustUltrafiltrationConfirmResponse'. -[sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationEditResponse.h:47] -> [sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationConfirmResponse.h:51]: (warning) The class 'MAdjustUltrafiltrationEditResponse' defines member variable with name '_data' also defined in its parent class 'MAdjustUltrafiltrationConfirmResponse'. -[sources/view/valarmstatus.h:37]: (style) Class 'VAlarmStatus' has a constructor with 1 argument that is not explicit. Such constructors should in general be explicit for type safety reasons. Using the explicit keyword in the constructor means some mistakes when using the class can be avoided. -[sources/view/valarmstatus.cpp:79]: (style) The function 'alarmPriorityText' is never used. -[sources/view/vtreatmentadjustmentultrafiltrationconfirm.cpp:48]: (style) The function 'doOptionDuration' is never used. -[sources/view/vtreatmentadjustmentultrafiltrationconfirm.cpp:41]: (style) The function 'doOptionRate' is never used. -[sources/view/vtreatmentadjustmentultrafiltrationstate.cpp:40]: (style) The function 'doPause' is never used. -[sources/view/vtreatmentadjustmentultrafiltrationstate.cpp:46]: (style) The function 'doResume' is never used. -[sources/utility/types.cpp:21]: (style) The function 'floatCompare' is never used. -[sources/utility/types.cpp:28]: (style) The function 'getBits' is never used. -[sources/view/valarmstatus.cpp:118]: (style) The function 'id' is never used. -[sources/storage/filehandler.cpp:73]: (style) The function 'read' is never used. -[sources/view/valarmstatus.cpp:129]: (style) The function 'requestOkay' is never used. -[sources/view/valarmstatus.cpp:138]: (style) The function 'requestSilence' is never used. -[sources/canbus/frameinterface.cpp:271]: (style) The function 'timerEvent' is never used. +[sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationEditResponse.h:67] -> [sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationConfirmResponse.h:61]: (warning) The class 'MAdjustUltrafiltrationEditResponse' defines member variable with name 'stringPrefix' also defined in its parent class 'MAdjustUltrafiltrationConfirmResponse'. +[sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationEditResponse.h:79] -> [sources/model/hd/adjustment/MTreatmentAdjustUltrafiltrationConfirmResponse.h:79]: (warning) The class 'MAdjustUltrafiltrationEditResponse' defines member variable with name '_data' also defined in its parent class 'MAdjustUltrafiltrationConfirmResponse'. +[sources/view/hd/alarm/VAlarmStatus.h:45]: (style) Class 'VAlarmStatus' has a constructor with 1 argument that is not explicit. Such constructors should in general be explicit for type safety reasons. Using the explicit keyword in the constructor means some mistakes when using the class can be avoided. +[sources/view/hd/alarm/VAlarmStatus.cpp:78]: (style) The function 'alarmPriorityText' is never used. +[sources/view/VEventSpy.cpp:126]: (style) The function 'doMouseReset' is never used. +[sources/view/hd/adjustment/VTreatmentAdjustmentUltrafiltrationConfirm.cpp:66]: (style) The function 'doOptionDuration' is never used. +[sources/view/hd/adjustment/VTreatmentAdjustmentUltrafiltrationState.cpp:54]: (style) The function 'doResume' is never used. +[sources/view/VEventSpy.cpp:169]: (style) The function 'doTouchReset' 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/view/hd/alarm/VAlarmStatus.cpp:117]: (style) The function 'id' is never used. +[sources/storage/FileHandler.cpp:95]: (style) The function 'read' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:128]: (style) The function 'requestOkay' is never used. +[sources/view/hd/alarm/VAlarmStatus.cpp:137]: (style) The function 'requestSilence' 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.