openlp-core team mailing list archive

[Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>]

Review: Approve
I prefer to put braces around all my if ... else statements, even if it's only one line.

One other thing, if you have an else, and then an if inside the else, but you only have that if, you can put the if right after the else. E.g.

if () {
  ....
}
else if () {
  ....
}
-- 
https://code.launchpad.net/~j-corwin/openlp/general/+merge/60845
Your team OpenLP Core is subscribed to branch lp:openlp.