gh-127413: allow to show specialized bytecode via `dis` CLI by picnixz · Pull Request #127414 · python/cpython · GitHub
Skip to content

gh-127413: allow to show specialized bytecode via dis CLI - #127414

Merged
iritkatriel merged 4 commits into
python:mainfrom
picnixz:feat/dis/adaptive-cli-127413
Dec 5, 2024
Merged

gh-127413: allow to show specialized bytecode via dis CLI#127414
iritkatriel merged 4 commits into
python:mainfrom
picnixz:feat/dis/adaptive-cli-127413

Conversation

@picnixz

@picnixz picnixz commented Nov 29, 2024

Copy link
Copy Markdown
Member

@iritkatriel

Copy link
Copy Markdown
Member

@picnixz

picnixz commented Dec 5, 2024

Copy link
Copy Markdown
Member Author

Are there any tests for dis on command line?

I couldn't find any in test_dis.py.

@iritkatriel

Copy link
Copy Markdown
Member

Are there any tests for dis on command line?

I couldn't find any in test_dis.py.

Please make an issue to add some.

@iritkatriel

Copy link
Copy Markdown
Member

There's a merge conflict, but otherwise LGTM.

Comment thread Doc/whatsnew/3.14.rst
* Add :data:`errno.EHWPOISON` error code.
(Contributed by James Roy in :gh:`126585`.)


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.

This line was added after I fixed the conflict. Since we put two blank lines between sections, I think it's ok to keep but I can revert it.

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.

(At some point, we'll need a copy-edit because the above sections are not splitted with double lines while the sections below are... But since most of the sections have two blank lines, it's easier to add them to places that lack them).

@iritkatriel
iritkatriel enabled auto-merge (squash) December 5, 2024 14:46
@iritkatriel
iritkatriel merged commit 67b9a53 into python:main Dec 5, 2024
@picnixz
picnixz deleted the feat/dis/adaptive-cli-127413 branch December 5, 2024 15:03
@bedevere-bot

Copy link
Copy Markdown

srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants