[issue19184] dis module has incorrect docs for RAISE_VARARGS

2013-10-15 Thread Tyler B
Tyler B added the comment: Here's a revised suggestion that has the order changed. I have additional concerns but please provide comment on this revision. Thanks # 2.7 "Raises an exception. argc indicates the number of parameters to the raise statement, ranging from 0 to 3. The para

[issue19184] dis module has incorrect docs for RAISE_VARARGS

2013-10-15 Thread Tyler B
Tyler B added the comment: One last edit: Looked at the code and found differences between 3.4 and 2.7. 2.7 has 4 exceptions that can be raised while 3.4 has 3 exceptions. I propose including the full list of parameters but describing the exceptions in a way that's less specific abou

[issue19184] dis module has incorrect docs for RAISE_VARARGS

2013-10-15 Thread Tyler B
Tyler B added the comment: I wanted to make an edit so here's my revised comment: Looked at the code and found differences between 3.4 and 2.7. 2.7 has 4 exceptions that can be raised while 3.4 has 3 exceptions. I propose removing the "list of parameters" from the docum

[issue19184] dis module has incorrect docs for RAISE_VARARGS

2013-10-14 Thread Tyler B
Tyler B added the comment: Looked at the code and found differences between 3.4 and 2.7. 2.7 has 3 exceptions that can be raised while 3.4 has 4 exceptions. I propose removing the "list of parameters" from the documenation to keep things simple and not repeat the code. # 2.7