code review 4518050: go/ast, go/doc, godoc: consider struct fields and inter...

go/ast, go/doc, godoc: consider struct fields and interface methods when filtering ASTs

So far, only top-level names where considered when trimming ASTs
using a filter function. For instance, "godoc reflect Implements"
didn't show the "Implements" method of the type Interface because
the local method name was not considered (on the other hand, "top-
level" declared methods associated with types were considered).

With this CL, AST filter functions look also at struct fields
and interface methods.

[gri, 2011-05-10 00:48:50]

Hello rsc, r (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[rsc, 2011-05-10 17:58:02]

LGTM

[r, 2011-05-10 17:59:15]

LGTM

[gri, 2011-05-10 18:10:01]

*** Submitted as http://code.google.com/p/go/source/detail?r=de003ba6a506 ***

go/ast, go/doc, godoc: consider struct fields and interface methods when
filtering ASTs

So far, only top-level names where considered when trimming ASTs
using a filter function. For instance, "godoc reflect Implements"
didn't show the "Implements" method of the type Interface because
the local method name was not considered (on the other hand, "top-
level" declared methods associated with types were considered).

With this CL, AST filter functions look also at struct fields
and interface methods.

R=rsc, r
CC=golang-dev
http://codereview.appspot.com/4518050