[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Raymond Hettinger]

Change by Raymond Hettinger :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Raymond Hettinger]

Raymond Hettinger  added the comment:


New changeset 8c21941ddafdf4925170f9cea22e2382dd3b0184 by Alexander Böhn in 
branch 'main':
bpo-39549: reprlib.Repr uses a “fillvalue” attribute (GH-18343)
https://github.com/python/cpython/commit/8c21941ddafdf4925170f9cea22e2382dd3b0184


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Raymond Hettinger]

Change by Raymond Hettinger :


--
assignee:  -> rhettinger

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Cheryl Sabella]

Change by Cheryl Sabella :


--
nosy: +rhettinger
versions:  -Python 2.7, Python 3.5, Python 3.6, Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Alexander Böhn]

Change by Alexander Böhn :


--
components: +Tests

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Alexander Böhn]

Change by Alexander Böhn :


--
keywords: +patch
pull_requests: +17716
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/18343

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39549] The reprlib.Repr type should permit the “fillvalue” to be set by the user

[Alexander Böhn]

New submission from Alexander Böhn :

Currently, the `reprlib.recursive_repr(…)` decorator allows a “fillvalue” 
parameter to be specified by the user. This is a string value that is used as a 
placeholder when calculating an objects’ repr – in the case of 
`recursive_repr(…)` the “fillvalue” defaults to '...' and may be set by the 
user to a string of any length.

There is no such user-defined “fillvalue” on the `reprlib.Repr` type, although 
the '...' string is hardcoded in its implementation and used throughout.

I propose that the hardcoded use of the '...' string in the code for the 
`reprlib.Repr` implementation should be replaced by a “fillvalue” attribute, 
set on the class in its `__init__(…)` method – and therefore overridable in 
subclasses, like the existing myriad “max*” instance attributes. 

PR to follow in short order.

--
components: Library (Lib)
messages: 361334
nosy: fish2000
priority: normal
severity: normal
status: open
title: The reprlib.Repr type should permit the “fillvalue” to be set by the user
type: behavior
versions: Python 2.7, Python 3.5, Python 3.6, Python 3.7, Python 3.8, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com