Which warnings do the two function trigger?
Also, adding { ... } breaks the code style.

the specific warning is https://clang.llvm.org/extra/clang-tidy/checks/readability-else-after-return.html

In D20398#446407, @pino wrote:

Which warnings do the two function trigger?
Also, adding { ... } breaks the code style.

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

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. :)