← 返回首页
bpo-36346: Make using the legacy Unicode C API optional by serhiy-storchaka · Pull Request #21437 · python/cpython · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted
Cancel Submit feedback

Saved searches

Use saved searches to filter your results more quickly

Cancel Create saved search
Appearance settings
Resetting focus

bpo-36346: Make using the legacy Unicode C API optional#21437

Merged
serhiy-storchaka merged 32 commits into
python:masterfrom
serhiy-storchaka:do-not-use-wchar-cache
Jul 10, 2020
Merged

bpo-36346: Make using the legacy Unicode C API optional#21437
serhiy-storchaka merged 32 commits into
python:masterfrom
serhiy-storchaka:do-not-use-wchar-cache

Conversation

serhiy-storchaka commented Jul 10, 2020 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Do not use the legacy Unicode C API which depends on the wchar_t* cache if USE_UNICODE_WCHAR_CACHE is defined as false.

It can help to test in advance the effect of removing the wchar_t* cache.

https://bugs.python.org/issue36346

bpo-36346: Prepare for removing the legacy Unicode C API. …
699e616
Add two compile time options: HAVE_UNICODE_WCHAR_CACHE and USE_UNICODE_WCHAR_CACHE. USE_UNICODE_WCHAR_CACHE makes the interpreter not using the wchar_t cache and the legacy Unicode C API. HAVE_UNICODE_WCHAR_CACHE removes the wchar_t cache and the legacy Unicode C API that depends on it.
Fix winreg.SetValue().
60b89c9
Clean up some ifdefs in _testcapimodule.
f386b63
Make path_cleanup() paranoidally safer.
11e0e0c
Merge branch 'master' into disable-wchar-cache
236f608
Fix os.scandir().
8750d48
Merge branch 'master' into disable-wchar-cache
431e71f
Merge branch 'master' into disable-wchar-cache
545c7a9
Merge branch 'master' into disable-wchar-cache
3c1ab31
Merge branch 'master' into disable-wchar-cache
54b0561
Merge branch 'master' into disable-wchar-cache
2c62d96
Merge branch 'master' into disable-wchar-cache
6d89775
Silence compiler warnings.
3b5294a
Fix PyUnicode_IsIdentifier for the cache-less build.
e7898fa
Silence compiler warnings on Windows.
fd641c6
Merge branch 'master' into disable-wchar-cache
a79d935
Fix compiler warning in _testcapi.
ed14aa9
Merge branch 'master' into disable-wchar-cache
c5eb102
Merge branch 'master' into disable-wchar-cache
d529224
Merge branch 'master' into disable-wchar-cache
6102b4b
Merge branch 'master' into disable-wchar-cache
74695b3
Use HAVE_UNICODE_WCHAR_CACHE instead of USE_UNICODE_WCHAR_CACHE in _P… …
97b5228
…y_DumpASCII().
Set HAVE_UNICODE_WCHAR_CACHE and USE_UNICODE_WCHAR_CACHE only if they… …
d6ba6b7
… were not set.
Remove Py_UNICODE_MATCH.
0da4146
Merge branch 'master' into disable-wchar-cache
51365fb
Fix unterminated #if.
e425908
Reset arraymodule.c.
8a9259b
Merge branch 'master' into disable-wchar-cache
76ba4b6
Temporary disable the wchar_t cache by default.
16ac7fd
Remove HAVE_UNICODE_WCHAR_CACHE.
5950b5c
serhiy-storchaka requested review from a team, rhettinger and skrah as code owners July 10, 2020 15:47
serhiy-storchaka merged commit 4c8f09d into python:master Jul 10, 2020
serhiy-storchaka deleted the do-not-use-wchar-cache branch July 10, 2020 20:26
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Aug 20, 2020
bpo-36346: Make using the legacy Unicode C API optional (pythonGH-21437) …
11ce089
Add compile time option USE_UNICODE_WCHAR_CACHE. Setting it to 0 makes the interpreter not using the wchar_t cache and the legacy Unicode C API.
xzy3 pushed a commit to xzy3/cpython that referenced this pull request Oct 18, 2020
bpo-36346: Make using the legacy Unicode C API optional (pythonGH-21437) …
14ade10
Add compile time option USE_UNICODE_WCHAR_CACHE. Setting it to 0 makes the interpreter not using the wchar_t cache and the legacy Unicode C API.
bedevere-bot mentioned this pull request Apr 10, 2022
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

rhettinger Awaiting requested review from rhettinger
skrah Awaiting requested review from skrah

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Footer

© 2026 GitHub, Inc.