Details
Details
- Reviewers
jgrulich crossi - Commits
- R116:c56f00b52786: Use more return early
Diff Detail
Diff Detail
- Repository
- R116 Plasma Network Management Applet
- Branch
- ret
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 18128 Build 18146: arc lint + arc unit
libs/models/networkmodel.cpp | ||
---|---|---|
402–404 | Code style : Use curly braces even when the body of a conditional statement contains only one line. |
libs/models/networkmodel.cpp | ||
---|---|---|
402–404 | I think this is a valid Qt coding style, it was just my obsession to use curly braces everywhere, even for single line body. |
libs/models/networkmodel.cpp | ||
---|---|---|
402–404 | That's a good obsession and help a lot to avoid error . ;) |
libs/models/networkmodel.cpp | ||
---|---|---|
402–404 | As much as I hate it, crossi is right. https://community.kde.org/Policies/Kdelibs_Coding_Style#Braces |