Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Apply github notifications and JSL loading enhancements from Fightwarn branch #2129

Merged
merged 2 commits into from
Oct 24, 2023

Jenkinsfile-dynamatrix: add a dynamatrixGlobalState.enableDebugTraceG…

971f53e
Select commit
Loading
Failed to load commit list.
Merged

CI: Apply github notifications and JSL loading enhancements from Fightwarn branch #2129

Jenkinsfile-dynamatrix: add a dynamatrixGlobalState.enableDebugTraceG…
971f53e
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
NUT CI farm - Jenkins / CppCheck Issues failed Oct 24, 2023 in 0s

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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins 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

See this annotation in the file changed.

@nut-ci-farm-jenkins nut-ci-farm-jenkins / CppCheck Issues

unreadVariable

LOW:
Variable 'response_timeout_ms' is assigned a value that is never used.