This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients larry, mdk, rhettinger, vstinner
Date 2016-11-25.08:29:46
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1480062586.76.0.68170859816.issue28754@psf.upfronthosting.co.za>
In-reply-to
Content
issue28754-4.diff does 3 things:
* convert C code to Argument Clinic
* change docstrings of the C code
* change unit tests

issue28754-5.diff doesn't touch tests, but still does 2 things. Sadly, I dislike the changes on docstrings because it makes the C docstring and the Python docstring inconsistent.

I suggest to copy/paste docstring between C and Python code.

By the way, why do we have a docstring different from Doc/library/bisect.rst? Maybe we can simply use the same doc everywhere, but just add reST formatting in Doc/library/bisect.rst?

Julien: I suggest to first restrict your change to Argument Clinic, and later write a new patch to update docstrings. I prefer to only do one thing per commit.
History
Date User Action Args
2016-11-25 08:29:46vstinnersetrecipients: + vstinner, rhettinger, larry, mdk
2016-11-25 08:29:46vstinnersetmessageid: <1480062586.76.0.68170859816.issue28754@psf.upfronthosting.co.za>
2016-11-25 08:29:46vstinnerlinkissue28754 messages
2016-11-25 08:29:46vstinnercreate