code review 157830044: compiler: Don't record interface types with blank type ...

compiler: Don't record interface types with blank type names. Fixes issue 8079.

[cmang, 2014-10-10 20:05:41]

Hello iant@golang.org (cc: gofrontend-dev@googlegroups.com, gri@golang.org), I'd like you to review this change to https://code.google.com/p/gofrontend

[cmang, 2014-10-10 20:08:22]

On 2014/10/10 20:05:41, cmang wrote: > Hello mailto:iant@golang.org (cc: mailto:gofrontend-dev@googlegroups.com, mailto:gri@golang.org), > > I'd like you to review this change to > https://code.google.com/p/gofrontend Should I add the test case from issue 8079 to the go testsuite?

[cmang, 2014-10-13 19:34:40]

On 2014/10/10 20:08:22, cmang wrote: > On 2014/10/10 20:05:41, cmang wrote: > > Hello mailto:iant@golang.org (cc: mailto:gofrontend-dev@googlegroups.com, > mailto:gri@golang.org), > > > > I'd like you to review this change to > > https://code.google.com/p/gofrontend > > Should I add the test case from issue 8079 to the go testsuite? This test case has been added to the main repository in https://code.google.com/p/go/source/detail?r=dbf133a972a5.

[iant, 2014-10-16 18:35:26]

This patch now conflicts with trunk. Please "hg sync" and then "hg upload 157830044". Thanks.

[cmang, 2014-10-16 18:42:22]

On 2014/10/16 18:35:26, iant wrote: > This patch now conflicts with trunk. Please "hg sync" and then "hg upload > 157830044". Thanks. Done.

[iant, 2014-10-16 19:27:49]

LGTM

[iant, 2014-10-16 19:28:18]

*** Submitted as https://code.google.com/p/gofrontend/source/detail?r=c401743ee460 *** compiler: Don't record interface types with blank type names. Fixes issue 8079. LGTM=iant R=iant CC=gofrontend-dev, gri https://codereview.appspot.com/157830044 Committer: Ian Lance Taylor <iant@golang.org>