Message45783
Logged In: YES
user_id=149084
1. Please improve the doc string for interesting_lines(),
it's no longer a generator.
2. You removed a comment from update_label() which
was helpful when studying the code. What does it do now?
3. The variable 'l' should not be used because it's hard
to discriminate from '1'. See PEP 8, Naming Conventions.
I suggest 'ln' if it must be short, otherwise 'line' |
|
Date |
User |
Action |
Args |
2007-08-23 15:37:13 | admin | link | issue936169 messages |
2007-08-23 15:37:13 | admin | create | |
|