code review 3988045: encoding/line: fix up a few typos and infelicities in t...

encoding/line: fix up a few typos and infelicities in the doc comments

[r, 2011-01-22 22:53:57]

Hello agl, anschelsc@gmail.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[anschelsc, 2011-01-23 04:48:53]

On 2011/01/22 22:53:57, r wrote:
> Hello agl,
https://mail.google.com/mail/?view=cm&fs=1&tf=1&to=anschelsc@gmail.com (cc:
https://mail.google.com/mail/?view=cm&fs=1&tf=1&to=golang-dev@googlegroups.com),
> 
> I'd like you to review this change.

While we're at it, should NewReader() have documentation? I know its function is
obvious, but the standard in most packages is to have something like "NewReader
returns a new Reader reading from r." (quoted from archive/tar)

[r, 2011-01-23 07:35:38]

Hello agl, anschelsc@gmail.com, anschelsc (cc: golang-dev@googlegroups.com),

Please take another look.

[anschelsc, 2011-01-23 13:37:38]

On 2011/01/23 07:35:38, r wrote:
> Hello agl,
https://mail.google.com/mail/?view=cm&fs=1&tf=1&to=anschelsc@gmail.com,
anschelsc (cc:
https://mail.google.com/mail/?view=cm&fs=1&tf=1&to=golang-dev@googlegroups.com),
> 
> Please take another look.

LGTM

[agl1, 2011-01-24 15:07:58]

LGTM, thanks!

http://codereview.appspot.com/3988045/diff/4001/src/pkg/encoding/line/line.go
File src/pkg/encoding/line/line.go (right):

http://codereview.appspot.com/3988045/diff/4001/src/pkg/encoding/line/line.go...
src/pkg/encoding/line/line.go:21: // NewReader returns a new reader that will
read successive
s/reader/Reader/ ?

http://codereview.appspot.com/3988045/diff/4001/src/pkg/encoding/line/line.go...
src/pkg/encoding/line/line.go:22: // lines from r.
should probably either change 'r' to 'in', or 'in' to 'r'.

[r, 2011-01-24 18:19:29]

*** Submitted as http://code.google.com/p/go/source/detail?r=163450d02f33 ***

encoding/line: fix up a few typos and infelicities in the doc comments

R=anschelsc, agl1
CC=golang-dev
http://codereview.appspot.com/3988045