code review 6845112: cmd/gc: merge casep and casee labels

cmd/gc: merge casep and casee labels

The code inside the casee and casep labels can perfectly be merged since
they essentially do the same. The character to be stored where cp points is
just the character contained by the c variable.

[mssola, 2012-11-29 21:34:22]

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

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

[dave_cheney.net, 2012-11-30 00:14:53]

On 2012/11/29 21:34:22, mssola wrote:
> Hello mailto:golang-dev@googlegroups.com (cc:
mailto:golang-dev@googlegroups.com),
> 
> I'd like you to review this change to
> https://code.google.com/p/go/

Please update the description to be

cmd/gc: merge casep and casee labels

The description should be phrased as what you want this change to do, not what
you did when making the change.

[mssola, 2012-11-30 07:46:19]

Hello golang-dev@googlegroups.com, dave@cheney.net (cc:
golang-dev@googlegroups.com),

Please take another look.

[rsc, 2012-12-06 06:46:53]

Looks good.
Please complete a CLA as described at
http://golang.org/doc/contribute.html#copyright.

Thanks.
Russ

[mssola, 2012-12-06 16:15:09]

Hi, according to http://golang.org/doc/contribute.html#copyright , since I'm not
a committer (I'm not in the Go's contributors file), I have to do nothing, but
the one who reviewed it will do the submitting. Right ? (notice that I'm a
first-timer here :D).

Greetings,
Miquel

[rsc, 2012-12-11 17:20:43]

LGTM

[rsc, 2012-12-11 17:23:13]

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

cmd/gc: merge casep and casee labels

The code inside the casee and casep labels can perfectly be merged since
they essentially do the same. The character to be stored where cp points is
just the character contained by the c variable.

R=golang-dev, dave, rsc
CC=golang-dev
https://codereview.appspot.com/6845112

Committer: Russ Cox <rsc@golang.org>