]> Cypherpunks repositories - gostls13.git/commitdiff
go/types, types2: remove need for Scope.LookupParent from TestObjectString
authorRobert Griesemer <gri@golang.org>
Fri, 27 Sep 2024 17:41:55 +0000 (10:41 -0700)
committerGopher Robot <gobot@golang.org>
Fri, 27 Sep 2024 18:18:44 +0000 (18:18 +0000)
Updates #69673.

Change-Id: I0ce5f009c1e95a2722a50d79a74fef83d2547b47
Reviewed-on: https://go-review.googlesource.com/c/go/+/616260
Auto-Submit: Robert Griesemer <gri@google.com>
Reviewed-by: Robert Griesemer <gri@google.com>
Reviewed-by: Alan Donovan <adonovan@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>

src/cmd/compile/internal/types2/object_test.go
src/go/types/object_test.go

index 955fa351a467dff0facf665b46f666f1734a75ed..f19264e4db6d41b039a7c1210b9b341f95639eeb 100644 (file)
@@ -129,10 +129,15 @@ func TestObjectString(t *testing.T) {
                        if len(names) != 1 && len(names) != 2 {
                                t.Fatalf("%s: invalid object path %s", test.src, test.obj)
                        }
-                       _, obj := pkg.Scope().LookupParent(names[0], nopos)
+
+                       var obj Object
+                       for s := pkg.Scope(); s != nil && obj == nil; s = s.Parent() {
+                               obj = s.Lookup(names[0])
+                       }
                        if obj == nil {
                                t.Fatalf("%s: %s not found", test.src, names[0])
                        }
+
                        if len(names) == 2 {
                                if typ, ok := obj.Type().(interface{ TypeParams() *TypeParamList }); ok {
                                        obj = lookupTypeParamObj(typ.TypeParams(), names[1])
index 704d4128ba86b3084baa07e253d3c5885edcfb54..0b4fce7bb161ceabcce2b75f17808b321e05db4f 100644 (file)
@@ -130,10 +130,15 @@ func TestObjectString(t *testing.T) {
                        if len(names) != 1 && len(names) != 2 {
                                t.Fatalf("%s: invalid object path %s", test.src, test.obj)
                        }
-                       _, obj := pkg.Scope().LookupParent(names[0], nopos)
+
+                       var obj Object
+                       for s := pkg.Scope(); s != nil && obj == nil; s = s.Parent() {
+                               obj = s.Lookup(names[0])
+                       }
                        if obj == nil {
                                t.Fatalf("%s: %s not found", test.src, names[0])
                        }
+
                        if len(names) == 2 {
                                if typ, ok := obj.Type().(interface{ TypeParams() *TypeParamList }); ok {
                                        obj = lookupTypeParamObj(typ.TypeParams(), names[1])