From: Dmitri Shuralyov <dmitshur@golang.org>
Date: Sat, 20 May 2023 03:20:43 +0000 (-0400)
Subject: cmd/dist: make two related panic messages more consistent
X-Git-Tag: go1.21rc1~284
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=c6f33ed11353620201fa2d4df40dd7304282539d;p=gostls13.git

cmd/dist: make two related panic messages more consistent

In contrast to the HasSuffix argument, there's no need or benefit in
having a ":" before the "racebench" variant mentioned in the message.
(The variant comes after the colon separator—it doesn't include it.)

Change-Id: Ie9948104de9449422037bf39245944255b98f1b5
Reviewed-on: https://go-review.googlesource.com/c/go/+/497735
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
---

diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go
index 849dad3640..3384149391 100644
--- a/src/cmd/dist/test.go
+++ b/src/cmd/dist/test.go
@@ -834,7 +834,7 @@ func (t *tester) addTest(name, heading string, fn func(*distTest) error) {
 	if !strings.Contains(name, ":") && heading != "Testing packages." {
 		panic("empty variant is reserved exclusively for registerStdTest")
 	} else if strings.HasSuffix(name, ":racebench") && heading != "Running benchmarks briefly." {
-		panic(":racebench variant is reserved exclusively for registerRaceBenchTest")
+		panic("racebench variant is reserved exclusively for registerRaceBenchTest")
 	}
 	if t.testNames == nil {
 		t.testNames = make(map[string]bool)