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

gh-141004: Document remaining PyCF compiler flag macros by johnslavik · Pull Request #153958 · python/cpython · GitHub

/ cpython Public

gh-141004: Document remaining PyCF compiler flag macros - #153958

Merged
ZeroIntensity merged 7 commits into
python:mainfrom
johnslavik:gh-141004-document-compile-flags
Jul 18, 2026
Merged

gh-141004: Document remaining PyCF compiler flag macros#153958
ZeroIntensity merged 7 commits into
python:mainfrom
johnslavik:gh-141004-document-compile-flags

Conversation

johnslavik commented Jul 18, 2026
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Document PyCF_DONT_IMPLY_DEDENT, PyCF_IGNORE_COOKIE, PyCF_SOURCE_IS_UTF8, PyCF_MASK and PyCF_COMPILE_MASK in the PyCompilerFlags section, with examples.

Document PyCF_DONT_IMPLY_DEDENT, PyCF_IGNORE_COOKIE,
PyCF_SOURCE_IS_UTF8, PyCF_MASK and PyCF_COMPILE_MASK in the
PyCompilerFlags section, with examples.

read-the-docs-community Bot commented Jul 18, 2026
edited
Loading

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #33646794 | 📁 Comparing 4551189 against main (0e81ef9)

  🔍 Preview build  

7 files changed · ± 7 modified

± Modified

ZeroIntensity left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

We should say somewhere in here that users should generally avoid these.

Comment thread Doc/c-api/veryhigh.rst
PyCF_DONT_IMPLY_DEDENT
PyCF_IGNORE_COOKIE
PyCF_MASK
PyCF_MASK_OBSOLETE

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Can we also document this one?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Clauding on that, yes

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

As asked, I documented PyCF_ALLOW_INCOMPLETE_INPUT (cc @pablogsal) and PyCF_MASK_OBSOLETE.

Both entries lead with a caution against using the flag. Remove the
two names from the C API docs ignore list.
Introduce them with a note that code outside the standard library
rarely needs them, and title the masks group for symmetry.
The masks serve the same narrow needs as the flags, so drop their
separate rubric and let the low-level introduction apply to both.

ZeroIntensity left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

github-project-automation Bot moved this from Todo to In Progress in Sprint Jul 18, 2026
ZeroIntensity added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels Jul 18, 2026
ZeroIntensity enabled auto-merge (squash) July 18, 2026 14:35
ZeroIntensity merged commit 0a772f2 into python:main Jul 18, 2026
49 checks passed
github-project-automation Bot moved this from Todo to Done in Docs PRs Jul 18, 2026
github-project-automation Bot moved this from In Progress to Done in Sprint Jul 18, 2026

Copy link
Copy Markdown

Thanks @johnslavik for the PR, and @ZeroIntensity for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

bedevere-app Bot commented Jul 18, 2026

Copy link
Copy Markdown

GH-153984 is a backport of this pull request to the 3.15 branch.

bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Jul 18, 2026

bedevere-app Bot commented Jul 18, 2026

Copy link
Copy Markdown

GH-153985 is a backport of this pull request to the 3.14 branch.

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Jul 18, 2026

bedevere-app Bot commented Jul 18, 2026

Copy link
Copy Markdown

GH-153986 is a backport of this pull request to the 3.13 branch.

bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Jul 18, 2026
ZeroIntensity pushed a commit that referenced this pull request Jul 18, 2026
…H-153958) (GH-153986)

(cherry picked from commit 0a772f2)

Co-authored-by: Bartosz Sławecki <bartosz@ilikepython.com>
ZeroIntensity pushed a commit that referenced this pull request Jul 18, 2026
…H-153958) (GH-153985)

(cherry picked from commit 0a772f2)

Co-authored-by: Bartosz Sławecki <bartosz@ilikepython.com>
ZeroIntensity pushed a commit that referenced this pull request Jul 18, 2026
…H-153958) (GH-153984)

(cherry picked from commit 0a772f2)

Co-authored-by: Bartosz Sławecki <bartosz@ilikepython.com>
johnslavik deleted the gh-141004-document-compile-flags branch July 21, 2026 01:55
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

docs Documentation in the Doc dir skip news sprint

Projects

Status: Done
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL