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

Issue 135170043: code review 135170043: test: add test that caused gccgo to crash on valid code (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 8 months ago by iant
Modified:
9 years, 8 months ago
Reviewers:
minux, dave
CC:
golang-codereviews, minux
Visibility:
Public.

Description

test: add test that caused gccgo to crash on valid code Update issue 8612

Patch Set 1 #

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+34 lines, -0 lines) Patch
A test/fixedbugs/issue8612.go View 1 chunk +34 lines, -0 lines 0 comments Download

Messages

Total messages: 4
iant
Hello golang-codereviews@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
9 years, 8 months ago (2014-08-29 02:39:32 UTC) #1
minux
LGTM.
9 years, 8 months ago (2014-08-29 02:40:45 UTC) #2
iant
*** Submitted as https://code.google.com/p/go/source/detail?r=765e304c1367 *** test: add test that caused gccgo to crash on valid ...
9 years, 8 months ago (2014-08-29 02:51:26 UTC) #3
dave_cheney.net
9 years, 8 months ago (2014-08-29 02:53:46 UTC) #4
Message was sent while issue was closed.
On 2014/08/29 02:40:45, minux wrote:
> LGTM.

Nice. Thanks Ian.
Sign in to reply to this message.

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