code review 6996046: go/ast: ast.DeclStmt.Decl must be an *ast.GenDecl node ...

go/ast: ast.DeclStmt.Decl must be an *ast.GenDecl node (documentation)

[gri, 2012-12-21 19:40:30]

Hello golang-dev@googlegroups.com,

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

[bradfitz, 2012-12-21 19:43:49]

LGTM
 On Dec 21, 2012 11:40 AM, <gri@golang.org> wrote:

> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go
>
>
> Description:
> go/ast: ast.DeclStmt.Decl must be an *ast.GenDecl node (documentation)
>
> Please review this at
https://codereview.appspot.**com/6996046/<https://codereview.appspot.com/6996...
>
> Affected files:
>   M src/pkg/go/ast/ast.go
>
>
> Index: src/pkg/go/ast/ast.go
> ==============================**==============================**=======
> --- a/src/pkg/go/ast/ast.go
> +++ b/src/pkg/go/ast/ast.go
> @@ -555,7 +555,7 @@
>
>         // A DeclStmt node represents a declaration in a statement list.
>         DeclStmt struct {
> -               Decl Decl
> +               Decl Decl // *GenDecl with CONST, TYPE, or VAR token
>         }
>
>         // An EmptyStmt node represents an empty statement.
>
>
>

[gri, 2012-12-21 19:52:23]

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

go/ast: ast.DeclStmt.Decl must be an *ast.GenDecl node (documentation)

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/6996046