Signed-off-by: André Agenor <andreagenor@icloud.com>
Details
Details
Diff Detail
Diff Detail
- Repository
- R374 KolourPaint
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Which warnings do the two function trigger?
Also, adding { ... } breaks the code style.
Comment Actions
the specific warning is https://clang.llvm.org/extra/clang-tidy/checks/readability-else-after-return.html
Comment Actions
This two warnings below
https://clang.llvm.org/extra/clang-tidy/checks/readability-else-after-return.html
https://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html
Sorry about code style, I tend to automate with clang format and I forgot the actual code style
Comment Actions
we don't have a clangformat file specified for kde projects, maybe this is
something good to add. I voted for global .editoconfigfiles but I was
outvoted. :)