[issue14824] reprlib documentation references string module

Berker Peksag added the comment: > I don't know why we would ever have a space in a typename, ever (and if > someone does awful hacks to get to that state, he should probably also do > awful hacks to make reprlib work properly). That part of the code has been added 22 years ago: https://hg.py

[issue14824] reprlib documentation references string module

Roundup Robot added the comment: New changeset e9968782c9ba by Berker Peksag in branch '3.4': Issue #14824: Update Repr.repr_TYPE documentation to use correct name mangling implementation. https://hg.python.org/cpython/rev/e9968782c9ba New changeset a0372781eafb by Berker Peksag in branch 'defa

[issue14824] reprlib documentation references string module

Changes by Berker Peksag : -- stage: -> patch review versions: +Python 3.4, Python 3.5 -Python 3.2, Python 3.3 ___ Python tracker ___ ___

[issue14824] reprlib documentation references string module

Chris Rebert added the comment: Updated patch. I concur that anyone creating a type whose name has whitespace in it is just asking for trouble. -- Added file: http://bugs.python.org/file27009/reprlib.rst.patch ___ Python tracker

[issue14824] reprlib documentation references string module

Jasper St. Pierre added the comment: The documentation is just flat out wrong, actually: if ' ' in typename: parts = typename.split() typename = '_'.join(parts) The documentation is claiming the inverse. I don't know why we would ever have a space in a typename, ever (and

[issue14824] reprlib documentation references string module

Changes by Éric Araujo : -- nosy: +eric.araujo versions: +Python 3.2, Python 3.3 -Python 3.4 ___ Python tracker ___ ___ Python-bugs-li

[issue14824] reprlib documentation references string module

Jasper St. Pierre added the comment: Yes. Yes it would. In my opinion, it really shouldn't do this sort of name mangling, as it's a terrible idea, but whatever. -- ___ Python tracker _

[issue14824] reprlib documentation references string module

Chris Rebert added the comment: Patch. Though I ponder whether the expression in question might be equivalent to simply: type(obj).__name__.replace('_', ' ') -- keywords: +patch nosy: +cvrebert Added file: http://bugs.python.org/file25607/reprlib.rst.patch

[issue14824] reprlib documentation references string module

New submission from Jasper St. Pierre : http://docs.python.org/dev/library/reprlib.html """ Formatting methods for specific types are implemented as methods with a name based on the type name. In the method name, TYPE is replaced by string.join(string.split(type(obj).__name__, '_')). Dispatch