code review 6294055: regexp/syntax: unexport ErrUnexpectedParen

regexp/syntax: unexport ErrUnexpectedParen

This new error is the only API change in the current draft of
Go 1.0.2 CLs. I'd like to include the CL that introduced it,
because it replaces a mysterious 'internal error' with a
useful error message, but I don't want any API changes,
so unexport the error constant for now. It can be
re-exported for Go 1.1.

[rsc, 2012-06-08 16:47:44]

Hello golang-dev@googlegroups.com,

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

[r, 2012-06-08 16:54:51]

LGTM

http://codereview.appspot.com/6294055/diff/2001/src/pkg/regexp/syntax/parse.go
File src/pkg/regexp/syntax/parse.go (right):

http://codereview.appspot.com/6294055/diff/2001/src/pkg/regexp/syntax/parse.g...
src/pkg/regexp/syntax/parse.go:51: const errUnexpectedParen ErrorCode =
"unexpected )"
// TODO: export for Go 1.1.

[rsc, 2012-06-08 17:05:08]

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

regexp/syntax: unexport ErrUnexpectedParen

This new error is the only API change in the current draft of
Go 1.0.2 CLs. I'd like to include the CL that introduced it,
because it replaces a mysterious 'internal error' with a
useful error message, but I don't want any API changes,
so unexport the error constant for now. It can be
re-exported for Go 1.1.

R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/6294055

[rsc, 2012-06-08 17:05:15]

On Fri, Jun 8, 2012 at 12:54 PM,  <r@golang.org> wrote:
> // TODO: export for Go 1.1.

done, and created issue 3712.