Review: Needs Fixing Line 8: The comment is too long. But you can simply omit the second sentence, because you already mention this in line 12. Line 12: Missing full stop. Line 13: You do not have to make the theme_list an instance variable. -- https://code.launchpad.net/~smpettit/openlp/ftw/+merge/70159 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
