LGTM - gri On Thu, Sep 29, 2011 at 1:28 PM, <iant@golang.org> wrote: > Reviewers: ...
13 years, 8 months ago
(2011-09-29 21:35:42 UTC)
#2
LGTM
- gri
On Thu, Sep 29, 2011 at 1:28 PM, <iant@golang.org> wrote:
> Reviewers: gri,
>
> Message:
> Hello gri (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> go spec: remove notes about gccgo limitations, now fixed
>
> Please review this at http://codereview.appspot.com/5164041/
>
> Affected files:
> M doc/go_spec.html
>
>
> Index: doc/go_spec.html
> ===================================================================
> --- a/doc/go_spec.html
> +++ b/doc/go_spec.html
> @@ -5265,7 +5265,5 @@
> <h2 id="Implementation_differences">Implementation differences - TODO</h2>
> <ul>
> <li><code>len(a)</code> is only a constant if <code>a</code> is a
> (qualified) identifier denoting an array or pointer to an array.</li>
> - <li>In gccgo, <code>nil</code> maps are not treated like empty
> maps.</li>
> - <li>In gccgo, trying to send/receive from a <code>nil</code> channel
> causes a run-time panic.</li>
> </ul>
> </span>
>
>
>
Issue 5164041: code review 5164041: go spec: remove notes about gccgo limitations, now fixed
(Closed)
Created 13 years, 8 months ago by iant
Modified 13 years, 8 months ago
Reviewers:
Base URL:
Comments: 0