feat: Adapt for Qt 6.8 #134
Annotations
10 warnings
cppcheck:
src/views/programmeritemwidget.cpp#L12
Parameter 'label' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.
|
cppcheck:
src/views/programmeritemwidget.cpp#L12
Parameter 'path' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.
|
cppcheck:
src/views/programmeritemwidget.h#L29
Class 'ProgrammerItemWidget' 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.
|
cppcheck:
src/views/programmeritemwidget.h#L28
Parameter 'path' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++.
|
cppcheck:
src/widgets/expressionbar.cpp#L560
Either the condition 'curpos==0' is redundant or exp.at() argument nr 1 can have invalid value. The value is -1 but the valid values are '0:'.
|
cppcheck:
src/widgets/proexpressionbar.cpp#L97
The scope of the variable 'funpos' 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.
|
cppcheck:
src/widgets/proexpressionbar.cpp#L98
The scope of the variable 'rightfunpos' 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.
|
cppcheck:
src/widgets/proexpressionbar.cpp#L97
Variable 'funpos' is assigned a value that is never used.
|
cppcheck:
src/widgets/proexpressionbar.cpp#L98
Variable 'rightfunpos' is assigned a value that is never used.
|
cppcheck:
src/widgets/proexpressionbar.cpp#L472
Local variable 'match' shadows outer variable
|