bpo-27200: fix configparser, copyreg and ctypes doctests by marco-buttu · Pull Request #240 · python/cpython

the-knights-who-say-ni


		.. testsetup::

		import configparser

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation should use three spaces here.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, you are right

		accessing it repeatedly returns the same object each time. On the other hand,
		accessing it through an index returns a new object each time:

		.. doctest::

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find your version much clearer, +1. However, I don't see much point converting this to a doctest. I'd say remove .. doctest:: and keep the rest of the changes.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed the doctest directive. However, there is one point for taking it: in some editors, if you use the :: before the example you enable the syntax hightligh for the example, but the example will not be doctested. If you use just the :, you lose the syntax highligh but the example will be doctested. If you eventually use the doctest directive, you have both syntax highlight and doctest.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, I didn't mentioned it because I assumed you are going to change "[...] each time:" to "[...] each time::" :)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@berkerpeksag, it was "[...] each time:" from the beginning, I did not change it. Do you still want to change it anyway? Sorry for all these questions :/