code review 5655047: unicode: various documentation tweaks

unicode: various documentation tweaks

The comment on IsOneOf regarding Latin-1 was an implementation detail:
when the function is called internally, that condition is true. It used to matter,
but now the comment is a dreg. The function works fine if the character is
Latin-1, so we just delete the comment.

Fixes issue 2966.

[r, 2012-02-10 03:21:13]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[bradfitz, 2012-02-10 03:29:02]

LGTM

http://codereview.appspot.com/5655047/diff/1/src/pkg/unicode/letter.go
File src/pkg/unicode/letter.go (right):

http://codereview.appspot.com/5655047/diff/1/src/pkg/unicode/letter.go#newcode63
src/pkg/unicode/letter.go:63: //BUG(r): There is no mechanism for full case
folding, that is, for
does BUG need to immediate proceed the //?

http://codereview.appspot.com/5655047/diff/1/src/pkg/unicode/letter.go#newcode64
src/pkg/unicode/letter.go:64: // characters  that involve multiple runes in the
input or output.
two spaces between characters and that

[r, 2012-02-10 03:30:51]

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

unicode: various documentation tweaks

The comment on IsOneOf regarding Latin-1 was an implementation detail:
when the function is called internally, that condition is true. It used to
matter,
but now the comment is a dreg. The function works fine if the character is
Latin-1, so we just delete the comment.

Fixes issue 2966.

R=golang-dev, bradfitz
CC=golang-dev
http://codereview.appspot.com/5655047