openlp-core team mailing list archive

[Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>]

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.