Details
Details
- Reviewers
jgrulich crossi - Commits
- R116:c56f00b52786: Use more return early
Diff Detail
Diff Detail
- Repository
- R116 Plasma Network Management Applet
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
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 |