Review: Approve If you wanted to, you could make the __init__ function of ThemeXML take named parameters so that it's a little cooler, like so:
theme = ThemeXML(parse=theme_xml) It would cut 2 lines down to 1. Just a nice-to-have though. -- https://code.launchpad.net/~trb143/openlp/ThemeManager2/+merge/5302 Your team openlp.org 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

