From: Daniel Martí Date: Thu, 30 May 2019 09:23:23 +0000 (+0100) Subject: cmd/go: add two more uses of lazyregexp X-Git-Tag: go1.13beta1~179 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=16796afc35de049a89da42efaad1593c26a96392;p=gostls13.git cmd/go: add two more uses of lazyregexp A couple of regexp.MustCompile globals have made their way in since we introduced this package for cmd/go. Convert the declarations. It's just two of them, so the cmd/go exec benchmark isn't affected noticeably. Change-Id: Ibd0615c99b6a049124a952c59978fd714c1b9615 Reviewed-on: https://go-review.googlesource.com/c/go/+/179358 Run-TryBot: Daniel Martí TryBot-Result: Gobot Gobot Reviewed-by: Bryan C. Mills Reviewed-by: Brad Fitzpatrick --- diff --git a/src/cmd/go/internal/modconv/dep.go b/src/cmd/go/internal/modconv/dep.go index f433300171..ccd1fc7b75 100644 --- a/src/cmd/go/internal/modconv/dep.go +++ b/src/cmd/go/internal/modconv/dep.go @@ -6,9 +6,9 @@ package modconv import ( "fmt" + "internal/lazyregexp" "net/url" "path" - "regexp" "strconv" "strings" @@ -96,7 +96,7 @@ func ParseGopkgLock(file string, data []byte) (*modfile.File, error) { return mf, nil } -var scpSyntaxReg = regexp.MustCompile(`^([a-zA-Z0-9_]+)@([a-zA-Z0-9._-]+):(.*)$`) +var scpSyntaxReg = lazyregexp.New(`^([a-zA-Z0-9_]+)@([a-zA-Z0-9._-]+):(.*)$`) func decodeSource(source string) (string, error) { var u *url.URL diff --git a/src/cmd/go/internal/sumweb/server.go b/src/cmd/go/internal/sumweb/server.go index 4619e9719b..5050805f87 100644 --- a/src/cmd/go/internal/sumweb/server.go +++ b/src/cmd/go/internal/sumweb/server.go @@ -7,9 +7,9 @@ package sumweb import ( "context" + "internal/lazyregexp" "net/http" "os" - "regexp" "strings" "cmd/go/internal/tlog" @@ -59,7 +59,7 @@ var Paths = []string{ "/tile/", } -var modVerRE = regexp.MustCompile(`^[^@]+@v[0-9]+\.[0-9]+\.[0-9]+(-[^@]*)?(\+incompatible)?$`) +var modVerRE = lazyregexp.New(`^[^@]+@v[0-9]+\.[0-9]+\.[0-9]+(-[^@]*)?(\+incompatible)?$`) func (h *Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) { ctx, err := h.Server.NewContext(r)