Support new booleans in Tkinter

[serhiy-storchaka]

BPO	21526
Nosy	@loewis, @amauryfa, @vstinner, @serhiy-storchaka
Files	tkinter_newboolean.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/serhiy-storchaka'
closed_at = <Date 2015-04-02.22:28:46.688>
created_at = <Date 2014-05-18.20:41:31.789>
labels = ['type-feature', 'expert-tkinter']
title = 'Support new booleans in Tkinter'
updated_at = <Date 2015-04-02.22:28:46.687>
user = 'https://github.com/serhiy-storchaka'

bugs.python.org fields:

activity = <Date 2015-04-02.22:28:46.687>
actor = 'serhiy.storchaka'
assignee = 'serhiy.storchaka'
closed = True
closed_date = <Date 2015-04-02.22:28:46.688>
closer = 'serhiy.storchaka'
components = ['Tkinter']
creation = <Date 2014-05-18.20:41:31.789>
creator = 'serhiy.storchaka'
dependencies = []
files = ['35284']
hgrepos = []
issue_num = 21526
keywords = ['patch']
message_count = 11.0
messages = ['218755', '238950', '239750', '239751', '239753', '239880', '239886', '239889', '239893', '239916', '239931']
nosy_count = 6.0
nosy_names = ['loewis', 'amaury.forgeotdarc', 'vstinner', 'gpolo', 'python-dev', 'serhiy.storchaka']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue21526'
versions = ['Python 3.5']

[serhiy-storchaka]

Recent Tcl versions (since 8.5) no longer use internal "boolean" type. Instead they use "booleanString" type (but looks as only internally). Here is a patch which add support for "booleanString" Tcl type in case when it leaks to user. There are no any new tests because there is no simple way (if any) to create pure Tcl boolean object.

[python-dev]

New changeset e9d86c1de292 by Serhiy Storchaka in branch 'default':
Issue bpo-21526: Tkinter now supports new boolean type in Tcl 8.5.
https://hg.python.org/cpython/rev/e9d86c1de292

[amauryfa]

I'm not sure that this change is correct.
I have Tcl version 8.6 installed, and I checked that "app->BooleanType" is NULL.

Fortunately value->typePtr is probably never NULL, but I think the comparison should be skipped in this case.

Where did you see this booleanString?

[serhiy-storchaka]

The case (value->typePtr == NULL) is tested above for purpose, so it is not a bug when one of type pointers is NULL.

"booleanString" was added in 8.5, replacing "boolean", but in 8.6 it is not used. All boolean values in 8.6 are just integers 0 and 1.

[amauryfa]

Ah, the NULL case is indeed tested just above. So the current code works correctly.

[serhiy-storchaka]

When I said this, I understood that the patch should be backported to maintained releases.

Unpatched Python with Tcl 8.5:

>>> import tkinter; tcl = tkinter.Tcl()
>>> tcl.call('expr', 'false')
<booleanString object: 'false'>
>>> bool(tcl.call('expr', 'false'))
True

[python-dev]

New changeset de3496cd609e by Serhiy Storchaka in branch '3.4':
Issue bpo-21526: Tkinter now supports new boolean type in Tcl 8.5.
https://hg.python.org/cpython/rev/de3496cd609e

New changeset b2413da7516f by Serhiy Storchaka in branch 'default':
Issue bpo-21526: Fixed support of new boolean type in Tcl 8.5.
https://hg.python.org/cpython/rev/b2413da7516f

New changeset d0554559de53 by Serhiy Storchaka in branch '2.7':
Issue bpo-21526: Tkinter now supports new boolean type in Tcl 8.5.
https://hg.python.org/cpython/rev/d0554559de53

[serhiy-storchaka]

And indeed, "app->BooleanType" is NULL, because the "booleanString" type is not registered in Tcl. Thank you for pointing on it Amaury. Now it is fixed.