homepage

The following code doesn't work. Would be great if we can fix it in 3.5
class Foo:
 __slots__ = ('__qualname__',)

Serhiy, maybe you know how to fix this?

Why should it work Yury?
__qualname__ and __doc__(if exists) are inserted into the dict when creating a class.
>>> class Foo:
... """bar"""
... __slots__ = ('__doc__',)
... 
Traceback (most recent call last):
 File "<stdin>", line 1, in <module>
ValueError: '__doc__' in __slots__ conflicts with class variable

Because the following works without a problem:
 class F:
 __slots__ = ('__name__', )
 f = F()
 f.__name__ = 'aaaa'
This bug with __qualname__ is why _GeneratorWrapper in types.py doesn't have __slots__.

What about other names set when creating a class? __module__, __class__, __classcell__?

I think of them but currently they don't pose a problem for practical codes like __qualname__. Maybe leaving them until there comes a real need?

> What about other names set when creating a class? __module__, __class__, __classcell__?
__module__ remains in the class dict so I think it's a class variable.
Will __class__ be set? It's inserted into the function scope.
__classcell__ I think has the same situation with __qualname__.
New patch also adds __classcell__.

slots_special_v2 fixes a bug in the previous patch.

__qualname__ doesn't makes much sense if it doesn't match __module__. If _GeneratorWrapper patches __qualname__, I think it should patch __module__ too.

v3 updates the test cases.

Nick, could you please look at the patch? Especially at the part about __classcell__.

"__classcell__" in __slots__ doesn't really make sense, as unlike __qualname__ (which turns into a real class level attribute and can be useful to override on instances), it's only a transient entry in a class definition namespace - types.__new__ pops it automatically so it doesn't get turned into an attribute.
However, consistency is a good thing, so allowing it seems reasonable.
Regarding the specifics, the pending patch on issue 23722 makes it a TypeError to ever set __classcell__ to anything other than a real cell object.

> However, consistency is a good thing, so allowing it seems reasonable.
The inconsistency also appears when "__classcell__" in slots with or without super().
v4 catches up with HEAD.

Xiang, can you make a PR?

PR made. :-)

New changeset c393ee858932f79bd6dabf31550f9a53ea90bc68 by Xiang Zhang in branch 'master':
bpo-24329: allow __qualname__ and __classcell__ in __slots__ (GH-495)
https://github.com/python/cpython/commit/c393ee858932f79bd6dabf31550f9a53ea90bc68