[issue16154] Some minor doc fixes in Doc/library

Ezio Melotti added the comment: Fixed, thanks for the report and the patch! -- assignee: docs@python -> ezio.melotti resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker

[issue16154] Some minor doc fixes in Doc/library

Roundup Robot added the comment: New changeset 9b3d0bdb9256 by Ezio Melotti in branch '2.7': #16154: fix some doctests in Doc/library. Patch by Ravi Sinha. http://hg.python.org/cpython/rev/9b3d0bdb9256 New changeset 5b405df8518d by Ezio Melotti in branch '3.2': #16154: fix some doctests in Doc/

[issue16154] Some minor doc fixes in Doc/library

Changes by Ezio Melotti : -- nosy: +ezio.melotti stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue16154] Some minor doc fixes in Doc/library

Ravi Sinha added the comment: Based on what I saw on some other issues, I think there is just a separate patch for each version (coming from separate working repositories), but all attached to the same issue. So I am doing the same. All the changes have been made to files under Doc/library/ -

[issue16154] Some minor doc fixes in Doc/library

Changes by Ravi Sinha : Added file: http://bugs.python.org/file27651/33.doc_lib_Oct21_2012.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue16154] Some minor doc fixes in Doc/library

Changes by Ravi Sinha : Added file: http://bugs.python.org/file27650/32.doc_lib_Oct21_2012.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue16154] Some minor doc fixes in Doc/library

Changes by Ravi Sinha : Added file: http://bugs.python.org/file27649/27.doc_lib_Oct21_2012.patch ___ Python tracker ___ ___ Python-bugs-list m

[issue16154] Some minor doc fixes in Doc/library

Ravi Sinha added the comment: I've used the testsetup directive and the problem in http://hg.python.org/cpython/file/40a1652349e9/Doc/library/math.rst, line 79, is fixed; the test passes Added # doctest: +SKIP to lines 179 and 180 in http://hg.python.org/cpython/file/40a1652349e9/Doc/library/

[issue16154] Some minor doc fixes in Doc/library

Terry J. Reedy added the comment: Apparently, I should have suggested "# doctest: +SKIP". The #12947 fix to example directives being deleted from the directive examples came after the 3.3.0 release. Fixed examples are at http://docs.python.org/py3k/library/doctest.html#option-flags-and-directive

[issue16154] Some minor doc fixes in Doc/library

Chris Jerdonek added the comment: > I considered the directive, but I mildly prefer the code change. I just wanted to point the directive out as an available option (especially for Ravi if he does further patches like this). Either way is fine with me. --

[issue16154] Some minor doc fixes in Doc/library

Terry J. Reedy added the comment: I considered the directive, but I mildly prefer the code change. end = ' ') guarantees a blank between fields even if one formats a too-large number end = '') does not add a blank to the end. end = '' if base == 'b' else ' ') does both (but depend

[issue16154] Some minor doc fixes in Doc/library

Terry J. Reedy added the comment: Now your patch: colorsys: same change all versions fractions: ditto Template: ditto I believe the changes I suggested for filecmp and the numbers should also be the same in all versions. ACKS: your name should be inserted in alpha order -- _

[issue16154] Some minor doc fixes in Doc/library

Chris Jerdonek added the comment: > Trailing blanks are *really* a problem The NORMALIZE_WHITESPACE doctest directive is another option to deal with cases like these. It ignores whitespace-only differences between the actual and expected outputs. Does it pass with that directive? --

[issue16154] Some minor doc fixes in Doc/library

Terry J. Reedy added the comment: I consider some items to be errors. In any case, all current docs can be fixed. First, your questions as to what you did not fix. fsum: I personally would add the import visibly, but at least add it hidden, as Chris suggested. 171 >>> from filecmp import dirc

[issue16154] Some minor doc fixes in Doc/library

Chris Jerdonek added the comment: > fsum will pass if 'from math import fsum' is done before it, but I've left > such issues for now since there seems to be a debate about how to go about > that For now, you can at least use the "testsetup" directive which will simultaneously allow the doctes

[issue16154] Some minor doc fixes in Doc/library

New submission from Ravi Sinha: - Doc/library/math.rst - fsum will pass if 'from math import fsum' is done before it, but I've left such issues for now since there seems to be a debate about how to go about that - some seem unfixable too - e.g. Doc/library/filecmp.rst - 'dir1' and 'dir