Message281685
| 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:46 | vstinner | set | recipients:
+ vstinner, rhettinger, larry, mdk |
| 2016年11月25日 08:29:46 | vstinner | set | messageid: <1480062586.76.0.68170859816.issue28754@psf.upfronthosting.co.za> |
| 2016年11月25日 08:29:46 | vstinner | link | issue28754 messages |
| 2016年11月25日 08:29:46 | vstinner | create |
|