]> Cypherpunks repositories - gostls13.git/commitdiff
go/types: remove pkgForMode in favor of pkgFor (cleanup)
authorRobert Griesemer <gri@golang.org>
Tue, 18 Oct 2022 20:00:26 +0000 (13:00 -0700)
committerGopher Robot <gobot@golang.org>
Tue, 18 Oct 2022 20:53:00 +0000 (20:53 +0000)
The mode is always 0.

Change-Id: I6566383c7724b26f070729041bce203a5afa0989
Reviewed-on: https://go-review.googlesource.com/c/go/+/443776
Reviewed-by: Robert Griesemer <gri@google.com>
Auto-Submit: Robert Griesemer <gri@google.com>
Run-TryBot: Robert Griesemer <gri@google.com>
Reviewed-by: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

src/go/types/api_test.go
src/go/types/instantiate_test.go
src/go/types/methodset_test.go
src/go/types/named_test.go

index b204025b5448d71342b9e5ff6b3118a56d739d7e..4874ab068c63b953467a214b1127e5b6478b00a0 100644 (file)
@@ -24,12 +24,8 @@ import (
 // pkgFor parses and type checks the package specified by path and source,
 // populating info if provided.
 func pkgFor(path, source string, info *Info) (*Package, error) {
-       return pkgForMode(path, source, info, 0)
-}
-
-func pkgForMode(path, source string, info *Info, mode parser.Mode) (*Package, error) {
        fset := token.NewFileSet()
-       f, err := parser.ParseFile(fset, path, source, mode)
+       f, err := parser.ParseFile(fset, path, source, 0)
        if err != nil {
                return nil, err
        }
@@ -1650,7 +1646,7 @@ func TestLookupFieldOrMethod(t *testing.T) {
        }
 
        for _, test := range tests {
-               pkg, err := pkgForMode("test", "package p;"+test.src, nil, 0)
+               pkg, err := pkgFor("test", "package p;"+test.src, nil)
                if err != nil {
                        t.Errorf("%s: incorrect test case: %s", test.src, err)
                        continue
@@ -1943,7 +1939,7 @@ func TestIdentical(t *testing.T) {
        }
 
        for _, test := range tests {
-               pkg, err := pkgForMode("test", "package p;"+test.src, nil, 0)
+               pkg, err := pkgFor("test", "package p;"+test.src, nil)
                if err != nil {
                        t.Errorf("%s: incorrect test case: %s", test.src, err)
                        continue
@@ -2241,7 +2237,7 @@ func f(x T) T { return foo.F(x) }
 func TestInstantiate(t *testing.T) {
        // eventually we like more tests but this is a start
        const src = "package p; type T[P any] *T[P]"
-       pkg, err := pkgForMode(".", src, nil, 0)
+       pkg, err := pkgFor(".", src, nil)
        if err != nil {
                t.Fatal(err)
        }
@@ -2279,7 +2275,7 @@ func TestInstantiateErrors(t *testing.T) {
 
        for _, test := range tests {
                src := "package p; " + test.src
-               pkg, err := pkgForMode(".", src, nil, 0)
+               pkg, err := pkgFor(".", src, nil)
                if err != nil {
                        t.Fatal(err)
                }
index 91dae734c17c7b37d705c11980885da4c1eca43e..ee68954a17774829a4f7f946b950b2b2cfb11981 100644 (file)
@@ -109,7 +109,7 @@ func TestInstantiateEquality(t *testing.T) {
        }
 
        for _, test := range tests {
-               pkg, err := pkgForMode(".", test.src, nil, 0)
+               pkg, err := pkgFor(".", test.src, nil)
                if err != nil {
                        t.Fatal(err)
                }
@@ -139,11 +139,11 @@ func TestInstantiateEquality(t *testing.T) {
 func TestInstantiateNonEquality(t *testing.T) {
        const src = "package p; type T[P any] int"
 
-       pkg1, err := pkgForMode(".", src, nil, 0)
+       pkg1, err := pkgFor(".", src, nil)
        if err != nil {
                t.Fatal(err)
        }
-       pkg2, err := pkgForMode(".", src, nil, 0)
+       pkg2, err := pkgFor(".", src, nil)
        if err != nil {
                t.Fatal(err)
        }
@@ -194,7 +194,7 @@ var X T[int]
 
        for _, test := range tests {
                src := prefix + test.decl
-               pkg, err := pkgForMode(".", src, nil, 0)
+               pkg, err := pkgFor(".", src, nil)
                if err != nil {
                        t.Fatal(err)
                }
@@ -219,7 +219,7 @@ func (T[P]) m() {}
 
 var _ T[int]
 `
-       pkg, err := pkgForMode(".", src, nil, 0)
+       pkg, err := pkgFor(".", src, nil)
        if err != nil {
                t.Fatal(err)
        }
index 610329ecb9bfd4d1ff73c9661a0b30e58d850b87..51303681a32ba66d26068c617751f63927ccf583 100644 (file)
@@ -84,7 +84,7 @@ func TestNewMethodSet(t *testing.T) {
        }
 
        check := func(src string, methods []method, generic bool) {
-               pkg, err := pkgForMode("test", "package p;"+src, nil, 0)
+               pkg, err := pkgFor("test", "package p;"+src, nil)
                if err != nil {
                        t.Errorf("%s: incorrect test case: %s", src, err)
                        return
index 0fe17418f400e96640f348e2c73f6185df62e597..945e25a41d9e085f1116a05bddaf6676d13fbc9a 100644 (file)
@@ -33,7 +33,7 @@ func (G[P]) N() (p P) { return }
 
 type Inst = G[int]
        `
-       pkg, err := pkgForMode("p", src, nil, 0)
+       pkg, err := pkgFor("p", src, nil)
        if err != nil {
                b.Fatal(err)
        }