From 78a608d785215364d16459c35a14690784438b0e Mon Sep 17 00:00:00 2001 From: "Bryan C. Mills" Date: Mon, 24 Feb 2020 09:21:04 -0500 Subject: [PATCH] cmd/go/internal/modfetch: delete unused isVendoredPackage function This function is apparently unused since CL 204917. Updates #35290 Updates #37397 Change-Id: Id7f5f5d5176fdbd1c5c6227e81d0854ceafc3f12 Reviewed-on: https://go-review.googlesource.com/c/go/+/220640 Run-TryBot: Bryan C. Mills TryBot-Result: Gobot Gobot Reviewed-by: Jay Conrod --- src/cmd/go/internal/modfetch/coderepo.go | 25 ------------------------ 1 file changed, 25 deletions(-) diff --git a/src/cmd/go/internal/modfetch/coderepo.go b/src/cmd/go/internal/modfetch/coderepo.go index d1d24a40c9..753f65b3de 100644 --- a/src/cmd/go/internal/modfetch/coderepo.go +++ b/src/cmd/go/internal/modfetch/coderepo.go @@ -1012,28 +1012,3 @@ func hasPathPrefix(s, prefix string) bool { return s[len(prefix)] == '/' && s[:len(prefix)] == prefix } } - -func isVendoredPackage(name string) bool { - var i int - if strings.HasPrefix(name, "vendor/") { - i += len("vendor/") - } else if j := strings.Index(name, "/vendor/"); j >= 0 { - // This offset looks incorrect; this should probably be - // - // i = j + len("/vendor/") - // - // (See https://golang.org/issue/31562.) - // - // Unfortunately, we can't fix it without invalidating checksums. - // Fortunately, the error appears to be strictly conservative: we'll retain - // vendored packages that we should have pruned, but we won't prune - // non-vendored packages that we should have retained. - // - // Since this defect doesn't seem to break anything, it's not worth fixing - // for now. - i += len("/vendor/") - } else { - return false - } - return strings.Contains(name[i:], "/") -} -- 2.48.1