chore: sync 1071develop #325
Merged
Annotations
9 warnings
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager-server/deepin-devicecontrol/src/drivercontrol/drivermanager.cpp#L109
Parameter 'errmsg' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager-server/deepin-devicecontrol/src/enablecontrol/enablesqlmanager.cpp#L25
Parameter 'strDriver' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager-server/deepin-deviceinfo/src/loadinfo/cpu/logicalcpu.cpp#L13
Class 'LogicalCpu' does not have a copy constructor which is recommended since it has dynamic memory/resource allocation(s).
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager-server/deepin-deviceinfo/src/loadinfo/cpu/logicalcpu.cpp#L13
Class 'LogicalCpu' does not have a operator= which is recommended since it has dynamic memory/resource allocation(s).
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager/src/SingleDeviceManager.cpp#L69
The function 'startDeviceManager' is never used.
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager/src/Tool/ThreadExecXrandr.cpp#L400
The scope of the variable 'curResolutionWidth' 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.
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager/src/Tool/ThreadExecXrandr.cpp#L400
The scope of the variable 'curResolutionHeight' 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.
|
Run linuxdeepin/action-cppcheck@main:
deepin-devicemanager/src/Widget/DetectedStatusWidget.cpp#L398
The function 'refreshUI' is never used.
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
The logs for this run have expired and are no longer available.
Loading