What summary says. Plus, just for the sake of readability I'd also suggest using if pos: instead of: if not pos == None: line above. They're equal, right?
(In reply to comment #0) > What summary says. Fixed, http://hg.mozilla.org/users/zbraniecki_mozilla.com/silme/rev/23d0b935550e > Plus, just for the sake of readability I'd also suggest using > if pos: > instead of: > if not pos == None: > line above. They're equal, right? Nah, if pos: ... catches pos=0 case.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Oh, right. Didn't think of that. Thanks for landing the patch.
You need to log in before you can comment on or make changes to this bug.