go.tools/go/types: LookupFieldOrMethod checks method set
LookupFieldOrMethod now also decides whether a found
method is actually in the method set. Simplifies call
sites. Added corresponding API tests.
TODO (separate CL): Decide what the correct value for
the indirect result should be (as required for code
generation). For now, the result value for indirect
is unchanged from before if a field/method is found.
Fixes issue 8584.
LGTM w.r.t. the status quo. https://codereview.appspot.com/132260043/diff/100001/go/types/api_test.go File go/types/api_test.go (right): https://codereview.appspot.com/132260043/diff/100001/go/types/api_test.go#newcode821 go/types/api_test.go:821: {"var x T; type ...
*** Submitted as https://code.google.com/p/go/source/detail?r=e964465ea698&repo=tools *** go.tools/go/types: LookupFieldOrMethod checks method set LookupFieldOrMethod now also decides whether ...
*** Submitted as
https://code.google.com/p/go/source/detail?r=e964465ea698&repo=tools ***
go.tools/go/types: LookupFieldOrMethod checks method set
LookupFieldOrMethod now also decides whether a found
method is actually in the method set. Simplifies call
sites. Added corresponding API tests.
TODO (separate CL): Decide what the correct value for
the indirect result should be (as required for code
generation). For now, the result value for indirect
is unchanged from before if a field/method is found.
Fixes issue 8584.
LGTM=adonovan
R=adonovan
CC=golang-codereviews
https://codereview.appspot.com/132260043
Issue 132260043: code review 132260043: go.tools/go/types: LookupFieldOrMethod checks method set
(Closed)
Created 10 years ago by gri
Modified 10 years ago
Reviewers:
Base URL:
Comments: 8