FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-156347: Fix the documentation of curses `window.encoding` by fedonman · Pull Request #156352 · python/cpython · GitHub

/ cpython Public

gh-156347: Fix the documentation of curses window.encoding - #156352

Open
fedonman wants to merge 1 commit into
python:mainfrom
fedonman:fix-curses-encoding-doc
Open

gh-156347: Fix the documentation of curses window.encoding#156352
fedonman wants to merge 1 commit into
python:mainfrom
fedonman:fix-curses-encoding-doc

Conversation

fedonman commented Aug 25, 2026
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

Reword the two places that describe window.encoding as encoding str arguments. On a wide build it is not used for that, so the HOWTO now names the encoding of the current locale, and the attribute's own entry scopes the claim to a narrow build and covers results as well as arguments.

Documentation only, so no NEWS fragment.

$ python -m sphinxlint Doc/howto/curses.rst Doc/library/curses.rst
No problems found.

The attribute is not used to encode str arguments on a build linked
against a wide-character version of the curses library, which is the
default: str is passed to curses as wide characters and the encoding of
the current locale determines the bytes.  It is used on a narrow build,
where it also decodes results.

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34220802 | 📁 Comparing b2faef2 against main (fe8ace3)

  🔍 Preview build  

2 files changed
± howto/curses.html
± library/curses.html

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL