← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #17056

Re: [Merge] lp:~trb143/openlp/bug-1022039 into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Needs Fixing

Lines 35 and 41 look to be the same test within each other.

(Seems very verbose, lots of elif's doing the same thing as the initial if!)
-- 
https://code.launchpad.net/~trb143/openlp/bug-1022039/+merge/123325
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next