Code review comment for lp:~evfool/checkbox/secondfix786924

Revision history for this message
Marc Tardif (cr3) wrote :

First, you don't need to declare and assign the count variable anymore. Second, for i in expression doesn't evaluate the expression on each iteration, so you don't need the findkey variable either. On a last note, I think that the for loop is pretty clear to not need such a large comment. I'll leave that to your discretion though.

review: Needs Fixing

« Back to merge proposal