[issue8899] Add docstrings to time.struct_time

2010-06-05 Thread Alexander Belopolsky
Alexander Belopolsky added the comment: Committed in r81756 (trunk) and r81757 (py3k). -- stage: commit review -> committed/rejected status: open -> closed ___ Python tracker ___

[issue8899] Add docstrings to time.struct_time

2010-06-04 Thread Alexander Belopolsky
Changes by Alexander Belopolsky : -- resolution: -> accepted ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue8899] Add docstrings to time.struct_time

2010-06-04 Thread Georg Brandl
Georg Brandl added the comment: The first change in time.rst is not needed; there the sentence is not a direct description of struct_time. The rest is good to commit. -- ___ Python tracker ___

[issue8899] Add docstrings to time.struct_time

2010-06-04 Thread Alexander Belopolsky
Alexander Belopolsky added the comment: Please see fixes in issue8899.diff. * Quite a few lines are pretty long. Please stay below 80 chars/line. Fixed. * There are trailing spaces in two member docs. Fixed. * The range notation is inconsistent. [first, last] or [first, last+1) is what w

[issue8899] Add docstrings to time.struct_time

2010-06-04 Thread Georg Brandl
Georg Brandl added the comment: Patch review: * Quite a few lines are pretty long. Please stay below 80 chars/line. * There are trailing spaces in two member docs. * The range notation is inconsistent. [first, last] or [first, last+1) is what we generally use. * "if cannot be determined" doe

[issue8899] Add docstrings to time.struct_time

2010-06-04 Thread Alexander Belopolsky
New submission from Alexander Belopolsky : The time.struct_time class is missing class and field docstrings: >>> time.struct_time.__doc__ is None True >>> time.struct_time.tm_year.__doc__ is None True This is significant because it is not obvious that field values are different from those of