X Tutup
The Wayback Machine - https://web.archive.org/web/20260216014428/https://github.com/python/cpython/pull/1159
Skip to content

bpo-19184: Fix dis module RAISE_VARARGS doc#1159

Closed
louisom wants to merge 2 commits intopython:masterfrom
louisom:fix_19184_dis_raise_varargs_docs
Closed

bpo-19184: Fix dis module RAISE_VARARGS doc#1159
louisom wants to merge 2 commits intopython:masterfrom
louisom:fix_19184_dis_raise_varargs_docs

Conversation

@louisom
Copy link
Contributor

@louisom louisom commented Apr 16, 2017

No description provided.

@mention-bot
Copy link

@lulouie, thanks for your PR! By analyzing the history of the files in this pull request, we identified @birkenfeld, @ncoghlan and @benjaminp to be potential reviewers.

Copy link
Member

@berkerpeksag berkerpeksag left a comment

Choose a reason for hiding this comment

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

I like Serhiy's suggestion at https://bugs.python.org/issue19184#msg291758 However, I'm not sure how it can be converted to prose. Perhaps @ncoghlan can give us some advice :)

@ncoghlan
Copy link
Contributor

The TOS and TOS1 convention that @serhiy-storchaka used in his suggestion is already common in the dis documentation, so that can just be converted directly to a bulleted list in the documentation. Perhaps something like:

Raises an exception. argc indicates the number of parameters pushed onto the stack, ranging from 0 to 2. These parameter counts correspond to the 3 different forms of the raise statement:

  • 0: raise (re-raise previous exception)
  • 1: raise TOS (raise exception instance or type at TOS)
  • 2: raise TOS1 from TOS (raise exception instance or type at TOS1 with __cause__ set to TOS)

@serhiy-storchaka serhiy-storchaka added the docs Documentation in the Doc dir label Dec 8, 2018
@csabella
Copy link
Contributor

I'm going to close this as it has an 'unknown repository'. Feel free to open a new pull request taking Nick's suggestions into account.

@csabella csabella closed this May 28, 2019
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

X Tutup