Created on 2013-09-16 08:47 by serhiy.storchaka, last changed 2013-09-23 20:08 by serhiy.storchaka. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| tkinter_repr.patch | serhiy.storchaka, 2013-09-16 08:47 | review | ||
| Messages (4) | |||
|---|---|---|---|
| msg197881 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) *  |
Date: 2013-09-16 08:47 | |
Currently the repr() for Tcl_Obj is not very useful. It exposes only Tcl type name and Tcl object address.
>>> import tkinter.ttk
>>> tv = tkinter.ttk.Treeview()
>>> tv.tag_configure('test', foreground='blue')
{}
>>> str(tv.tag_configure('test', 'foreground'))
'blue'
>>> tv.tag_configure('test', 'foreground')
<color object at 0xb70f84b8>
You need explicitly call str to get more useful information. This is awkward when Tcl object hidden deeply in tuples or other data structure, or reported in a backtrace. Actually every Tcl object can be represented as a string. I propose to expose this representation in repr(). With proposed patch the output of the last command will be: <color object: 'blue'>.
|
|||
| msg198237 - (view) | Author: Terry J. Reedy (terry.reedy) * |
Date: 2013-09-22 00:45 | |
This looks more useful to me; it might make tests easier. But I cannot be sure no one would object. |
|||
| msg198239 - (view) | Author: Raymond Hettinger (rhettinger) * |
Date: 2013-09-22 01:19 | |
+1 |
|||
| msg198331 - (view) | Author: Roundup Robot (python-dev) | Date: 2013-09-23 19:49 | |
New changeset ece634a69ba8 by Serhiy Storchaka in branch 'default': Issue #19034: repr() for tkinter.Tcl_Obj now exposes string reperesentation. http://hg.python.org/cpython/rev/ece634a69ba8 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2013-09-23 20:08:13 | serhiy.storchaka | set | status: open -> closed assignee: serhiy.storchaka resolution: fixed stage: patch review -> resolved |
| 2013-09-23 19:49:23 | python-dev | set | nosy:
+ python-dev messages: + msg198331 |
| 2013-09-22 01:19:59 | rhettinger | set | nosy:
+ rhettinger messages: + msg198239 |
| 2013-09-22 00:45:18 | terry.reedy | set | messages: + msg198237 |
| 2013-09-16 08:47:32 | serhiy.storchaka | create | |