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.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to