CI: Apply github notifications and JSL loading enhancements from Fightwarn branch #2129
12 new issues, 795 total
Total | New | Outstanding | Fixed | Trend |
---|---|---|---|---|
795 | 12 | 783 | 777 | 👍 |
Reference build: Network UPS Tools project dynamatrix » nut » fightwarn #109
Details
Severity distribution of new issues
Error | Warning High | Warning Normal | Warning Low |
---|---|---|---|
0 | 0 | 0 | 12 |
Annotations
Check warning on line 718 in drivers/dummy-ups.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'ptr' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 496 in drivers/sms_ser.c
nut-ci-farm-jenkins / CppCheck Issues
multiCondition
LOW:
Expression is always true because 'else if' condition is opposite to previous condition at line 484.. else if condition is opposite to first condition
Check warning on line 363 in drivers/sms_ser.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'battery_status' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 334 in drivers/tripplite_usb.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'ret' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 335 in drivers/tripplite_usb.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'u_msg' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 336 in drivers/tripplite_usb.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'u_value' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 0 in drivers/apc_modbus.c
nut-ci-farm-jenkins / CppCheck Issues
toomanyconfigs
LOW:
Too many #ifdef configurations - cppcheck only checks 12 of 23 configurations. Use --force to check all configurations.. The checking of the file will be interrupted because there are too many #ifdef configurations. Checking of all #ifdef configurations can be forced by --force command line option or from GUI preferences. However that may increase the checking time.
Check warning on line 1149 in drivers/apc_modbus.c
nut-ci-farm-jenkins / CppCheck Issues
redundantInitialization
LOW:
Redundant initialization for 'end'. The initialized value is overwritten before it is read.. end is overwritten
Check warning on line 328 in drivers/apc_modbus.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'res' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 823 in drivers/apc_modbus.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'v' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 1144 in drivers/apc_modbus.c
nut-ci-farm-jenkins / CppCheck Issues
variableScope
LOW:
The scope of the variable 'r' 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:\012void f(int x)\012{\012 int i = 0;\012 if (x) {\012 // it's safe to move 'int i = 0;' here\012 for (int n = 0; n < 10; ++n) {\012 // it is possible but not safe to move 'int i = 0;' here\012 do_something(&i);\012 }\012 }\012}\012When you see this message it is always safe to reduce the variable scope 1 level.
Check warning on line 1318 in drivers/apc_modbus.c
nut-ci-farm-jenkins / CppCheck Issues
unreadVariable
LOW:
Variable 'response_timeout_ms' is assigned a value that is never used.