Review: Approve
You could combine those last few lines:

    if lastimage is None:
        return False
    if not os.path.isfile(lastimage):
        return False

becomes:

    if lastimage is None or not os.path.isfile(lastimage):
        return False

-- 
https://code.launchpad.net/~j-corwin/openlp/present/+merge/13679
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