]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/go: add two more uses of lazyregexp
authorDaniel Martí <mvdan@mvdan.cc>
Thu, 30 May 2019 09:23:23 +0000 (10:23 +0100)
committerBrad Fitzpatrick <bradfitz@golang.org>
Thu, 30 May 2019 15:38:43 +0000 (15:38 +0000)
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í <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/cmd/go/internal/modconv/dep.go
src/cmd/go/internal/sumweb/server.go

index f43330017168ebe9cf0fb4c80d62f67cf19fe241..ccd1fc7b7594ff6af0e2d2d90c207cbcb7afed94 100644 (file)
@@ -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
index 4619e9719b8e95d7d3b4ae731262ffe8187665ce..5050805f873b8462562fe704ee472466828051de 100644 (file)
@@ -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)