Review: Needs Fixing

29      + for tryCount in range(3):
44      + if tryCount >= 2:
81      + for tryCount in range(3):
96      + if tryCount >= 2:
121     + for tryCount in range(3):
131     + if tryCount >= 2:
149     + for tryCount in range(3):
159     + if tryCount >= 2:
177     + for tryCount in range(3):
187     + if tryCount >= 2:
205     + for tryCount in range(3):
215     + if tryCount >= 2:
243     + for tryCount in range(3):
257     + if tryCount >= 2:
298     + for tryCount in range(3):
315     + if tryCount >= 2:

That should be "try_count". Local variables should use the PEP8 naming 
convention/syntax.
-- 
https://code.launchpad.net/~dave-warnock/openlp/dbclean/+merge/109546
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