code review 59210044: unicode: delete appearance of unused LowerUpper term fr...

unicode: delete appearance of unused LowerUpper term from comment
If a LowerUpper ever happens, maketables will complain.

Fixes issue 7002.

[r, 2014-01-31 23:08:16]

Hello golang-codereviews@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go

[dave_cheney.net, 2014-01-31 23:08:59]

LGTM. Thank you.


On Sat, Feb 1, 2014 at 10:08 AM, <r@golang.org> wrote:

> Reviewers: golang-codereviews,
>
> Message:
> Hello golang-codereviews@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go
>
>
> Description:
> unicode: delete appearance of unused LowerUpper term from comment
> Fixes issue 7002.
>
> Please review this at https://codereview.appspot.com/59210044/
>
> Affected files (+1, -1 lines):
>   M src/pkg/unicode/letter.go
>
>
> Index: src/pkg/unicode/letter.go
> ===================================================================
> --- a/src/pkg/unicode/letter.go
> +++ b/src/pkg/unicode/letter.go
> @@ -74,7 +74,7 @@
>
>  type d [MaxCase]rune // to make the CaseRanges text shorter
>
> -// If the Delta field of a CaseRange is UpperLower or LowerUpper, it means
> +// If the Delta field of a CaseRange is UpperLower, it means
>  // this CaseRange represents a sequence of the form (say)
>  // Upper Lower Upper Lower.
>  const (
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>

[r, 2014-01-31 23:10:04]

*** Submitted as https://code.google.com/p/go/source/detail?r=731cd4eca133 ***

unicode: delete appearance of unused LowerUpper term from comment
If a LowerUpper ever happens, maketables will complain.

Fixes issue 7002.

LGTM=dave
R=golang-codereviews, dave
CC=golang-codereviews
https://codereview.appspot.com/59210044

[gobot, 2014-01-31 23:13:39]

This CL appears to have broken the linux-amd64 builder.

[dave_cheney.net, 2014-01-31 23:16:18]

Known issue, https://code.google.com/p/go/issues/detail?id=7205


On Sat, Feb 1, 2014 at 10:13 AM, <gobot@golang.org> wrote:

> This CL appears to have broken the linux-amd64 builder.
>
> https://codereview.appspot.com/59210044/
>