Message151340
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. |
|
Date |
User |
Action |
Args |
2012-01-16 08:25:37 | terry.reedy | set | recipients:
+ terry.reedy, gpolo, ezio.melotti, roger.serwy, Freiddie, python-dev |
2012-01-16 08:25:37 | terry.reedy | set | messageid: <1326702337.36.0.372435287057.issue6528@psf.upfronthosting.co.za> |
2012-01-16 08:25:36 | terry.reedy | link | issue6528 messages |
2012-01-16 08:25:36 | terry.reedy | create | |
|