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

gh-145854: Replace `DELETE_NAME` with `PUSH_NULL; STORE_NAME` by brijkapadia · Pull Request #146006 · python/cpython · GitHub

/ cpython Public

gh-145854: Replace DELETE_NAME with PUSH_NULL; STORE_NAME - #146006

Merged
markshannon merged 20 commits into
python:mainfrom
brijkapadia:delete-name
Jun 30, 2026
ConversationCommits (20)ChecksFiles changed
Merged

gh-145854: Replace DELETE_NAME with PUSH_NULL; STORE_NAME#146006
markshannon merged 20 commits into
python:mainfrom
brijkapadia:delete-name

Conversation

brijkapadia commented Mar 16, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

Changes

  • Removed DELETE_NAME
  • When STORE_NAME receives a NULL value, it corresponds with deletion

📚 Documentation preview 📚: https://cpython-previews--146006.org.readthedocs.build/

pythongh-145854: Replace DELETE_NAME with PUSH_NULL; STORE_NAME
1f97a49 
Changes
- Removed `DELETE_NAME`
- When `STORE_NAME` receives a `NULL` value, it corresponds with
  deletion
bedevere-app Bot mentioned this pull request Mar 16, 2026
Closed
brijkapadia mentioned this pull request Mar 16, 2026
Open
brijkapadia marked this pull request as draft March 16, 2026 02:31
brijkapadia marked this pull request as ready for review March 16, 2026 10:39

brijkapadia commented Mar 16, 2026

Copy link
Copy Markdown
Contributor Author

I am not sure why three of the CIFuzz tests fail. If anyone could help, that would be appreciated.

picnixz reviewed Mar 16, 2026
View reviewed changes
Comment thread Python/bytecodes.c Outdated
picnixz reviewed Mar 16, 2026
View reviewed changes
Comment thread Python/bytecodes.c Outdated

markshannon commented Mar 16, 2026

Copy link
Copy Markdown
Member

This will need a magic number bump (as do all changes that add or remove opcodes)

brijkapadia mentioned this pull request Mar 17, 2026
Closed
markshannon approved these changes Mar 17, 2026
View reviewed changes

markshannon 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

Looks good. Thanks for doing this.

brijkapadia commented Mar 27, 2026

Copy link
Copy Markdown
Contributor Author

@markshannon A gentle reminder about this PR.

kumaraditya303 commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

A gentle reminder about this PR.

There are merge conflicts.

python-cla-bot Bot commented Apr 17, 2026 •
edited
Loading

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

markshannon commented May 1, 2026

Copy link
Copy Markdown
Member

Sorry for leaving this sitting for so long.

I notice that _STORE_NAME is no longer suitable for jitting and is also quite large, so would bloat the stencils if it were.
Can you factor out the body of it into a _PyEval_StoreName() helper function?

read-the-docs-community Bot commented May 29, 2026 •
edited
Loading

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #33374710 | 📁 Comparing 87c2898 against main (2a07ff9)

  🔍 Preview build  

254 files changed · + 6 added · ± 247 modified · - 1 deleted

+ Added

± Modified

- Deleted

brijkapadia requested a review from markshannon May 29, 2026 16:52

brijkapadia commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

@markshannon A reminder about this PR, whenever you get a chance

markshannon requested changes Jun 30, 2026
View reviewed changes

markshannon 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

Looks good. Thanks for doing this.

I mistake in the magic number comment and needs merge conflicts fixing.
Otherwise, good to merge.

Comment thread Include/internal/pycore_magic_number.h Outdated

bedevere-app Bot commented Jun 30, 2026

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

brijkapadia commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again

bedevere-app Bot commented Jun 30, 2026

Copy link
Copy Markdown

Thanks for making the requested changes!

@markshannon: please review the changes made to this pull request.

bedevere-app Bot requested a review from markshannon June 30, 2026 13:44
markshannon approved these changes Jun 30, 2026
View reviewed changes

markshannon 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

Thanks

markshannon merged commit c393ab6 into python:main Jun 30, 2026
83 of 86 checks passed
brijkapadia deleted the delete-name branch June 30, 2026 16:33
markshannon mentioned this pull request Jun 30, 2026
Merged
maurycy added a commit to maurycy/cpython that referenced this pull request Jul 1, 2026
Merge branch 'main' into maurycy/bye-abba
da7d768 
* main: (266 commits)
  pythongh-151626: Fix tests that fail when PYTHONPYCACHEPREFIX is set (pythonGH-151952)
  pythongh-152728: IDLE - move 3 toplevel fix_xyz functions to idlelb.util (python#152729)
  pythongh-152711: Add pythoninfo-build command to Platforms/Android (python#152713)
  pythongh-152715: Add pythoninfo-build command to Platforms/Apple (python#152716)
  pythongh-152433: Windows: enable mmapmodule for UWP (python#152473)
  pythongh-152433: Windows: use GetFileSizeEx instead of GetFileSize for memory mapped files (python#152383)
  pythonGH-81881: Raise `SpecialFileError` for sockets and devices in `shutil.copyfile` (python#142693)
  pythongh-152502: Detect the curses mouse interface and is_* methods portably (pythonGH-152705)
  pythongh-145857: Replace `DELETE_GLOBAL` with `PUSH_NULL; STORE_GLOBAL` (pythonGH-146314)
  pythongh-145854: Replace `DELETE_NAME` with `PUSH_NULL; STORE_NAME` (pythonGH-146006)
  pythongh-152680: Detect container/VM in test.pythoninfo (python#152668)
  pythongh-152682: Fix NULL dereference on OOM in `symtable_visit_type_param_bound_or_default` (python#152684)
  pythongh-151881: Skip tk_inactive negativity check on Windows (pythonGH-152683)
  pythongh-152546: Refactor `mappingproxy.__new__` to use `PyDictProxy_New` (python#152547)
  pythongh-151126: Fix a possible crash during the startup with no memory under `Py_STACKREF_DEBUG` (python#152478)
  pythongh-152635: Raise MemoryError when the lock allocation fails in `_interpchannels.create()` (python#152642)
  pythongh-151029: Fix `test_remote_exec_deleted_static_executable` on static installed builds (pythonGH-152653)
  pythongh-121249: Deprecate using F/D type codes in the struct module (python#152309)
  pythongh-152192: Fix JUMP_BACKWARD passing a truncated oparg to the jit tracer (pythonGH-152382)
  Don't require the `_test{internal}capi` modules in `test_monitoring.py` (python#152311)
  ...
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

picnixz picnixz left review comments
kumaraditya303 kumaraditya303 left review comments
markshannon markshannon approved these changes
ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently is a code owner
iritkatriel Awaiting requested review from iritkatriel iritkatriel is a code owner
AA-Turner Awaiting requested review from AA-Turner
StanFromIreland Awaiting requested review from StanFromIreland

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL