From: James Yang <26634873@qq.com> Date: Tue, 23 May 2023 02:52:55 +0000 (+0000) Subject: cmd/go/internal/modfetch: fix retractions slice initial length not zero X-Git-Tag: go1.21rc2~1^2~6 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=bc21d6a4fc;p=gostls13.git cmd/go/internal/modfetch: fix retractions slice initial length not zero When make slice of retractions, it should have initial length zero, to append more VersionIntervals. Currently without the zero length, the capacity used will be doubled after the appending, looks like a bug. Change-Id: Id3acaeffe557ca1d15c864b0377a66fee3a41f6c GitHub-Last-Rev: ed5fd5f6784c43195c531d8acc75560adff104ba GitHub-Pull-Request: golang/go#60354 Reviewed-on: https://go-review.googlesource.com/c/go/+/497118 Auto-Submit: Bryan Mills Reviewed-by: Bryan Mills Run-TryBot: Bryan Mills TryBot-Result: Gopher Robot Reviewed-by: Dmitri Shuralyov --- diff --git a/src/cmd/go/internal/modfetch/coderepo.go b/src/cmd/go/internal/modfetch/coderepo.go index 50f4bb2b37..8fe432a9f5 100644 --- a/src/cmd/go/internal/modfetch/coderepo.go +++ b/src/cmd/go/internal/modfetch/coderepo.go @@ -1013,7 +1013,7 @@ func (r *codeRepo) retractedVersions(ctx context.Context) (func(string) bool, er if err != nil { return nil, err } - retractions := make([]modfile.VersionInterval, len(f.Retract)) + retractions := make([]modfile.VersionInterval, 0, len(f.Retract)) for _, r := range f.Retract { retractions = append(retractions, r.VersionInterval) }