Terry J. Reedy <[email protected]> added the comment:
I just realized that there is 'another' reason not to reuse builtin names
(other than those usually given): they will be colorized as builtins even if
they have no relation to the builtin. Make_pat uses the function any() defined
just above (before the builtin, I am sure), so 'any' is colored as if it were
the builtin. Within ColorDelegator.py, it is only used within make_pat, so I
considered renaming it to anyp or any_pat, but it might be used by some other
module that imports ColorDelegator. Another issue.
"name not in keyword.kwlist" could be "name not in {None, True, False}", but
those happen to be the first three items in the kwlist, so the time difference
would be nil.
Looks good. Tested on 3.2.2 Win7. Applied.
----------
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue6528>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
