Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in

Issue 4358054: Issue 11670

Can't Edit
Can't Publish+Mail
Start Review
13 years, 3 months ago by Lukasz.Langa (ambv)
13 years, 2 months ago
Larry Hastings

Patch Set 1 #

Total comments: 5
Unified diffs Side-by-side diffs Delta from patch set Stats (+71 lines, -9 lines) Patch
M Doc/library/configparser.rst View 1 chunk +13 lines, -5 lines 3 comments Download
M Lib/configparser.py View 1 chunk +4 lines, -4 lines 1 comment Download
M Lib/test/test_cfgparser.py View 2 chunks +54 lines, -0 lines 1 comment Download


Total messages: 2
Larry Hastings
Sorry for the delay, but I'm in the process of moving to a new house ...
13 years, 3 months ago (2011-04-10 07:04:45 UTC) #1
Lukasz.Langa (ambv)
13 years, 2 months ago (2011-04-27 15:45:10 UTC) #2
On 2011/04/10 07:04:45, Larry Hastings wrote:
> Doc/library/configparser.rst:979: files.
> 80 cols please.

> Your phrasing is a bit awkward.  May I suggest:
> Read and parse configuration data from *f*, which must be an iterable yielding
> Unicode strings (for example any file object or file-like object).

Thanks for help. Definitely sounds better.

> Doc/library/configparser.rst:989: be migrated by wrapping them with the
> following generator::
> Again your phrasing isn't good English; I'm guessing you're not a native
> speaker.

No, I'm not. Didn't think it stands out so much, though ;)

> Prior to Python 3.2, this function was called :meth:`readfp`, and it read
> from *f* by calling the object's :meth:`readline` method.  If you have
> code that calls :meth:`readfp`, and the object you pass in as *f* supports
> :meth:`readline` but doesn't support iteration, you can migrate to Python 3.2
> wrapping your object with the following generator::

This is inaccurate and misses the style used in the rest of the document. I
rewrote it once again, hope the result meets your taste as well :)

> Doc/library/configparser.rst:995: line = f.readline()
> Do you want to give an example of use here?

Sure, good idea.

> http://codereview.appspot.com/4358054/diff/1/Lib/configparser.py#newcode697
> Lib/configparser.py:697: The `f' argument must be iterable, returning one line
> at a time.
> Your text is okay, but I think this is a little better:
> The 'f' argument must be iterable, behaving identically to a file-like object.

This version is less specific. I'd rather keep my wording this time.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b