Looks fine to me. Would prefer an actual new test, but I suppose I can let updated tests slide...
I want to just do some testing on both macOS and Linux before I'll approve, but it's too late right now, I'll look at it tomorrow (Sunday). -- https://code.launchpad.net/~springermac/openlp/media-fixes/+merge/327480 Your team OpenLP Core is requested to review the proposed merge of lp:~springermac/openlp/media-fixes into 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