Review: Approve approve
Looks fine. Just remember that log.[exception|debug|warn|etc] don't need %, use 
commas:

    log.debug('My variable: %s', myvariable)
-- 
https://code.launchpad.net/~meths/openlp/trivialfixes/+merge/28017
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to