Review: Needs Fixing
Blank line police will get you with blink lines inside methods.
Log.debug strings are not set up with unicode.

124,127,233 line length may be others.

310,318,320 comment should be on line above and others
-- 
https://code.launchpad.net/~derek-scotney/openlp/songselectfileimport/+merge/34128
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