Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1779)

Issue 127950043: code review 127950043: compiler: comma-ok assignments produce untyped bool as ... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by cmang
Modified:
10 years, 7 months ago
Reviewers:
gobot, rsc, iant
CC:
gri, rsc, golang-codereviews
Visibility:
Public.

Description

cmd/gc: comma-ok assignments produce untyped bool as 2nd result

Patch Set 1 #

Patch Set 2 : diff -r 0449858880beeb4216dc94ac59892205473f6f78 https://code.google.com/p/go #

Patch Set 3 : diff -r 0449858880beeb4216dc94ac59892205473f6f78 https://code.google.com/p/go #

Patch Set 4 : diff -r 0449858880beeb4216dc94ac59892205473f6f78 https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+47 lines, -7 lines) Patch
M src/cmd/gc/order.c View 1 1 chunk +4 lines, -1 line 0 comments Download
M src/cmd/gc/typecheck.c View 1 1 chunk +1 line, -1 line 0 comments Download
M src/cmd/gc/walk.c View 1 3 chunks +13 lines, -1 line 0 comments Download
A test/fixedbugs/issue8475.go View 1 1 chunk +25 lines, -0 lines 0 comments Download
M test/named1.go View 1 1 chunk +4 lines, -4 lines 0 comments Download

Messages

Total messages: 7
cmang
Hello gri@golang.org, rsc@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
10 years, 7 months ago (2014-08-11 22:40:27 UTC) #1
rsc
change prefix to "cmd/gc" not compiler
10 years, 7 months ago (2014-08-11 22:48:59 UTC) #2
cmang
On 2014/08/11 22:48:59, rsc wrote: > change prefix to "cmd/gc" not compiler Fixed
10 years, 7 months ago (2014-08-11 22:51:25 UTC) #3
rsc
LGTM Thanks.
10 years, 7 months ago (2014-08-11 22:59:32 UTC) #4
cmang
*** Submitted as https://code.google.com/p/go/source/detail?r=3a2621a2e43b *** cmd/gc: comma-ok assignments produce untyped bool as 2nd result LGTM=rsc ...
10 years, 7 months ago (2014-08-11 23:11:59 UTC) #5
gobot
This CL appears to have broken the windows-386 builder. See http://build.golang.org/log/e522dcefeddcb1db5f647f6e3c5f6cb039b04fb4
10 years, 7 months ago (2014-08-11 23:38:03 UTC) #6
iant
10 years, 7 months ago (2014-08-11 23:41:15 UTC) #7
On Mon, Aug 11, 2014 at 4:38 PM,  <gobot@golang.org> wrote:
> This CL appears to have broken the windows-386 builder.
> See http://build.golang.org/log/e522dcefeddcb1db5f647f6e3c5f6cb039b04fb4
>
>
> https://codereview.appspot.com/127950043/

Nope.

--- FAIL: TestConnICMPEcho (0.27s)
ipraw_test.go:133: Conn.Read failed: WSARecv ip4 127.0.0.1: i/o timeout
FAIL
FAIL net 18.905s
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b