Message73108
self.deletecommand doesn't remove menu item, so we don't have to care
about index shifting like bellow. +1 for gpolo's patch.
>>> a = [0, 1, 2, 3]
>>> for i in xrange(len(a)):
... del a[i]
...
Traceback (most recent call last):
File "<stdin>", line 2, in <module>
IndexError: list assignment index out of range
I'm not sure (self._tclCommands is not None) check is not really needed.
I was looking for the place self._tclCommands is initialized, and found
_register() is that place, but what is 'needcleanup'? :-0
But probably, gpolo's patch is right.
P.S.
This is not related to this issue, I think
"""Delete menu items between INDEX1 and INDEX2 (not included)."""
should be changed to
"""Delete menu items between INDEX1 and INDEX2 (included)."""
Please look at http://www.tcl.tk/man/tcl8.5/TkCmd/menu.htm#M59 |
|
Date |
User |
Action |
Args |
2008-09-12 17:33:30 | ocean-city | set | recipients:
+ ocean-city, loewis, svenil, benjamin.peterson, gpolo, schuppenies, skomoroh, indiedan |
2008-09-12 17:33:30 | ocean-city | set | messageid: <1221240810.88.0.321661698971.issue3774@psf.upfronthosting.co.za> |
2008-09-12 17:33:30 | ocean-city | link | issue3774 messages |
2008-09-12 17:33:29 | ocean-city | create | |
|