Skip to content

gh-97928: Change the behavior of tkinter.Text.count() #98484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 24, 2023

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Oct 20, 2022

It now always returns an integer if one or less counting options are specified. Previously it could return a single count as a 1-tuple, an integer (only if option "update" was specified) or None if no items found. The result is now the same if wantobjects is set to 0.

It now always returns an integer if one or less counting options are specified.
Previously it could return a single count as a 1-tuple, an integer (only if
option "update" was specified) or None if no items found.
The result is now the same if wantobjects is set to 0.
@serhiy-storchaka
Copy link
Member Author

It is a breaking change. It broke a code in IDLE which contained a workaround for the current behavior, but the fixed code is clearer.

@serhiy-storchaka serhiy-storchaka marked this pull request as ready for review October 24, 2023 09:05
@serhiy-storchaka serhiy-storchaka merged commit b8c20f9 into python:main Oct 24, 2023
@serhiy-storchaka serhiy-storchaka deleted the tkinter-text-count branch October 24, 2023 09:59
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Feb 5, 2024
…) by default

By default, it preserves an inconsistent behavior of older Python
versions: packs the count into a 1-tuple if only one or none
options are specified (including 'update').
Except that setting wantobjects to 0 no longer affects the result.

Add a new parameter return_ints: specifying return_ints=True makes
Text.count() returning the single count as an integer.
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…-98484)

It now always returns an integer if one or less counting options are specified.
Previously it could return a single count as a 1-tuple, an integer (only if
option "update" was specified) or None if no items found.
The result is now the same if wantobjects is set to 0.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…-98484)

It now always returns an integer if one or less counting options are specified.
Previously it could return a single count as a 1-tuple, an integer (only if
option "update" was specified) or None if no items found.
The result is now the same if wantobjects is set to 0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants