openlp-core team mailing list archive

[Tim Bentley <tim.bentley@xxxxxxxxx>]

Review: Needs Fixing

29-31 should be 10 lines higher with the rest of the header statements

49-50 etc should be on the same line
Line 69 one more indent.

Line 80 far too long!

82-85 why is this commented out

134-135 indents look wrong

138-145 styling wrong (indents and spaces)

174, 179 indents and spaces wrong
-- 
https://code.launchpad.net/~marmyshev/openlp/autoplay/+merge/140289
Your team OpenLP Core is subscribed to branch lp:openlp.