Changes by Mark Lawrence breamore...@yahoo.co.uk:
--
nosy: -BreamoreBoy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4037
___
___
Filip Zyzniewski added the comment:
It seems like there is also an issue with property classes defined in a
different module.
In the example below Foo.x uses standard property, and Foo.y uses prop imported
from the prop module.
This results in docstring for Foo.y to be missed:
Changes by R. David Murray rdmur...@bitdance.com:
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4037
___
___
Changes by Terry J. Reedy tjre...@udel.edu:
--
versions: -Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4037
___
___
Python-bugs-list
Mark Lawrence breamore...@yahoo.co.uk added the comment:
Could someone please respond to Georg's comment msg104300, thanks.
--
nosy: +BreamoreBoy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4037
Georg Brandl ge...@python.org added the comment:
So for staticmethods and classmethods, valname doesn't need to be reassigned?
--
nosy: +georg.brandl
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4037
Steven D'Aprano steve+pyt...@pearwood.info added the comment:
I have fixed the issue with line length, and taken Brian's advice re valname.
Updated patch for doctest and test.test_doctest2 is attached.
--
Added file: http://bugs.python.org/file16599/doctest_patch
Changes by Nick Coghlan ncogh...@gmail.com:
--
nosy: +ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4037
___
___
Python-bugs-list mailing
Steven D'Aprano steve+pyt...@pearwood.info added the comment:
Attached is a simple test script for the patch I submitted. I have tested it
with Python 2.6 both before and after applying the patch. Run it from the
command line.
With the unpatched doctest module, it prints:
Expected 2
Brian Curtin cur...@acm.org added the comment:
Can you add your test(s) in Lib/test/test_doctest.py ? That way it will be run
with the Python regression suite. Ideally a documentation update would come
with the patch. Also, line length should ideally be capped at 79 characters
(re: PEP-8).
Steven D'Aprano steve+pyt...@pearwood.info added the comment:
The patch you suggest is *not* sufficient, at least not by my testing.
However, the attached patch does work, according to my tests.
--
nosy: +stevenjd
Added file: http://bugs.python.org/file16158/patch
Terry J. Reedy tjre...@udel.edu added the comment:
I am not sure whether this would be considered a bugfix or a new feature.
(ie, whether it would apply to 2.6 and 3.1 or not)
But the change is needed for 3.x also.
I am not sure whether the doc needs changing. A testcase should be added to
New submission from daaku [EMAIL PROTECTED]:
doctest.py currently does not include doctests from method descriptors
in a class.
The patch is simple, in the _find function in class DocTestFinder:
Original:
# Recurse to methods, properties, and nested classes.
if
13 matches
Mail list logo