Message387244
Victor
> the new globals has no "__builtins__" key. This change is backward incompatible on purpose. If someone really wants to run a function in a different builtins namespace, globals['__builtins__'] must be set explicitly.
You say it's on purpose, what's the purpose? Aren't you worried this is going to break stuff? And why is this necessary given the LOAD_GLOBAL cache?
Yury
> this hypothetical LEN opcode would still have to check if "len" was shadowed or not, and that's slower than the optimized LOAD_GLOBAL we have now.
It could use the same check though? Just check the version tags. |
|
Date |
User |
Action |
Args |
2021-02-18 16:56:27 | gvanrossum | set | recipients:
+ gvanrossum, brett.cannon, rhettinger, vstinner, petr.viktorin, Mark.Shannon, serhiy.storchaka, yselivanov |
2021-02-18 16:56:27 | gvanrossum | set | messageid: <1613667387.15.0.26721585315.issue42990@roundup.psfhosted.org> |
2021-02-18 16:56:27 | gvanrossum | link | issue42990 messages |
2021-02-18 16:56:27 | gvanrossum | create | |
|