Mirroristas/etcd
2
0
Fork 0
mirror of https://github.com/etcd-io/etcd.git synced 2024-09-27 06:25:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #17652 from callthingsoff/make_Regexp_global

Browse Source 
all: make regexp.Regexp variables global when not using configuration…
This commit is contained in:
Benjamin Wang
2024-03-27 16:52:05 +00:00
committed by GitHub
parent 647b7eb8f6 f3daf5e041
commit f1836fca99
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
client/pkg/testutil/leak.go
View File

@@ -34,6 +34,8 @@ running(leaking) after all tests.
...
}
*/
var normalizedRegexp = regexp.MustCompile(`\(0[0-9a-fx, ]*\)`)
func CheckLeakedGoroutine() bool {
gs := interestingGoroutines()
if len(gs) == 0 {
@@ -41,10 +43,9 @@ func CheckLeakedGoroutine() bool {
}
stackCount := make(map[string]int)
re := regexp.MustCompile(`\(0[0-9a-fx, ]*\)`)
for _, g := range gs {
// strip out pointer arguments in first function of stack dump
normalized := string(re.ReplaceAll([]byte(g), []byte("(...)")))
normalized := string(normalizedRegexp.ReplaceAll([]byte(g), []byte("(...)")))
stackCount[normalized]++
}

5
etcdctl/ctlv3/command/util.go
View File

@@ -56,9 +56,10 @@ func addHexPrefix(s string) string {
return string(ns)
}
var argsRegexp = regexp.MustCompile(`"(?:[^"\\]|\\.)*"|'[^']*'|[^'"\s]\S*[^'"\s]?`)
func Argify(s string) []string {
r := regexp.MustCompile(`"(?:[^"\\]|\\.)*"|'[^']*'|[^'"\s]\S*[^'"\s]?`)
args := r.FindAllString(s, -1)
args := argsRegexp.FindAllString(s, -1)
for i := range args {
if len(args[i]) == 0 {
continue