]> Cypherpunks repositories - gostls13.git/commitdiff
go/types: remove TODOs from API that cannot be changed (cleanup)
authorRobert Griesemer <gri@golang.org>
Thu, 14 Dec 2017 00:40:39 +0000 (16:40 -0800)
committerRobert Griesemer <gri@golang.org>
Thu, 14 Dec 2017 00:45:50 +0000 (00:45 +0000)
Change-Id: I4e357a250b2f9cfbf780b05c5f54805682b19a62
Reviewed-on: https://go-review.googlesource.com/83935
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
src/go/types/type.go

index ee7159f2e232498936654a75ddc57515c13c7d28..a58684a535fd4604d4dc4be0040d40452799ef4a 100644 (file)
@@ -418,7 +418,6 @@ func (t *Named) NumMethods() int { return len(t.methods) }
 func (t *Named) Method(i int) *Func { return t.methods[i] }
 
 // SetUnderlying sets the underlying type and marks t as complete.
-// TODO(gri) determine if there's a better solution rather than providing this function
 func (t *Named) SetUnderlying(underlying Type) {
        if underlying == nil {
                panic("types.Named.SetUnderlying: underlying type must not be nil")
@@ -430,7 +429,6 @@ func (t *Named) SetUnderlying(underlying Type) {
 }
 
 // AddMethod adds method m unless it is already in the method list.
-// TODO(gri) find a better solution instead of providing this function
 func (t *Named) AddMethod(m *Func) {
        if i, _ := lookupMethod(t.methods, m.pkg, m.name); i < 0 {
                t.methods = append(t.methods, m)