◐ Shell
reader mode source ↗
Skip to content

[3.11] gh-94823: Improve coverage in tokenizer.c:valid_utf8 (GH-94856)#96029

Merged
pablogsal merged 1 commit into
python:3.11from
miss-islington:backport-f215d7c-3.11
Aug 16, 2022
Merged

[3.11] gh-94823: Improve coverage in tokenizer.c:valid_utf8 (GH-94856)#96029
pablogsal merged 1 commit into
python:3.11from
miss-islington:backport-f215d7c-3.11

Conversation

@miss-islington

@miss-islington miss-islington commented Aug 16, 2022

Copy link
Copy Markdown
Contributor

When loading a source file from disk, there is a separate UTF-8 validator
distinct from the one in unicode_decode_utf8. This exercises that code path
with the same set of invalid inputs as we use for testing the "other" UTF-8
decoder.
(cherry picked from commit f215d7c)

Co-authored-by: Michael Droettboom mdboom@gmail.com

…94856)

When loading a source file from disk, there is a separate UTF-8 validator
distinct from the one in `unicode_decode_utf8`. This exercises that code path
with the same set of invalid inputs as we use for testing the "other" UTF-8
decoder.
(cherry picked from commit f215d7c)

Co-authored-by: Michael Droettboom <mdboom@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

Status check is done, and it's a success ✅ .

@pablogsal pablogsal merged commit 2bb363c into python:3.11 Aug 16, 2022
@miss-islington

Copy link
Copy Markdown
Contributor Author

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

@miss-islington miss-islington deleted the backport-f215d7c-3.11 branch August 16, 2022 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants