You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Every symbol table has type and name. The name of the symbol table that corresponds a class or a function is the same as the name of the corresponding class or function. But there are special symbol tables of type function for lambdas and generator expressions (there were also symbol tables for comprehensions, but they are no longer used). For lambdas. the name "lambda" does not conflict with other function names, because it is a reserved word. But for general expressions, the name "genexpr" can conflict with local function "genexpr" (see #119698). It is possible to distinguish the symbol table corresponding to a generator expression by looking in the list of its parameters, but this is not so convenient.
I propose to make the difference more clear:
Either use separate types for lambda and generator expression symbol tables.
Or use names which cannot be confused with any function name:
reserved word like lambda. But I do not know good variant for generator expression.
hyphenated word, like gen-expr.
angle brackets, like <genexpr> and <lambda>.
other special characters, e.g. .genexpr.
Names of symbol tables of other types like top and __annotations__ can also be changed for uniformity and to avoid possible future conflicts.
The original issue #119698 was solved in other way, so there is no urge for such change. This is just a wild idea.
The text was updated successfully, but these errors were encountered:
I think a leading dot is a good approach to get a name that doesn't conflict with user-defined names: it always works and it's already what we do in a few other places in the compiler pipeline (e.g., PEP 695 annotation scopes can have a compiler-created local variable called .type_params).
Yes and I dislike that way although I couldn't come up with something else... It's a bit too fragile IMO =/
I'm actually happy with both proposals:
different type: you can have checks of the form .type == TYPE_GENERATOR_EXPRESSION without exposing any name. The issue is that you introduce more types but I'm not sure whether something else needs to be changed in symtable.c.
special names: I like the dotted names and since they are already part of PEP 695, maybe it's fine to extend their usage. Having a reserved word is kind of introducing a new keyword and I'm pretty sure that you that you don't want genexpr to be a hard keyword (or any other kind of name, like maybe __Py_genexpr but I don't think we gain anything here...). For brackets, I think it's already the case for the repr() but not in the symbol table itself.
Uh oh!
There was an error while loading. Please reload this page.
Feature or enhancement
Every symbol table has type and name. The name of the symbol table that corresponds a class or a function is the same as the name of the corresponding class or function. But there are special symbol tables of type function for lambdas and generator expressions (there were also symbol tables for comprehensions, but they are no longer used). For lambdas. the name "lambda" does not conflict with other function names, because it is a reserved word. But for general expressions, the name "genexpr" can conflict with local function "genexpr" (see #119698). It is possible to distinguish the symbol table corresponding to a generator expression by looking in the list of its parameters, but this is not so convenient.
I propose to make the difference more clear:
Either use separate types for lambda and generator expression symbol tables.
Or use names which cannot be confused with any function name:
lambda
. But I do not know good variant for generator expression.gen-expr
.<genexpr>
and<lambda>
..genexpr
.Names of symbol tables of other types like
top
and__annotations__
can also be changed for uniformity and to avoid possible future conflicts.The original issue #119698 was solved in other way, so there is no urge for such change. This is just a wild idea.
The text was updated successfully, but these errors were encountered: