Larry Hastings added the comment:

I made a couple final tweaks to what was essentially Nick's patch:

* Argument Clinic wraps parameters in the signature at 72 columns
  instead of 79 columns.  There are a couple extra characters that
  get emitted, so this ensures that the generated lines are never > 80.

* Added an assert to typeobject.c that end - start > 2, so that 
  the subsequent assert on end[-1] is always a legal memory access.

* inspect._strip_non_python_syntax() now emits a space after commas in
  the signature, enhancing readability on the off chance that someone
  needs to read them.

* Added a couple extra tests for _strip_non_python_syntax().

Thanks for all the reviews and the fixes, everybody!

----------
resolution:  -> fixed
stage: patch review -> committed/rejected
status: open -> closed

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue20530>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to