From: Rob Pike <r@golang.org>
Date: Thu, 22 Dec 2011 19:08:51 +0000 (-0800)
Subject: testing: fix wording in explanation of potential bug
X-Git-Tag: weekly.2011-12-22~22
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=34139ee155e303e13b8a648965840dd9ed907b4e;p=gostls13.git

testing: fix wording in explanation of potential bug
TBR=rsc

R=rsc
CC=golang-dev
https://golang.org/cl/5504075
---

diff --git a/src/pkg/testing/testing.go b/src/pkg/testing/testing.go
index b1fb8dd3de..0411a5f480 100644
--- a/src/pkg/testing/testing.go
+++ b/src/pkg/testing/testing.go
@@ -240,9 +240,9 @@ func RunTests(matchString func(pat, str string) (bool, error), tests []InternalT
 		runtime.GOMAXPROCS(procs)
 		// We build a new channel tree for each run of the loop.
 		// collector merges in one channel all the upstream signals from parallel tests.
-		// If all tests pump to the same channel, a bug can occur where a goroutine
-		// kicks off a test, fails, and still delivers a completion signal, which skews the
-		// counting.
+		// If all tests pump to the same channel, a bug can occur where a test
+		// kicks off a gouroutine that Fails, yet the test still delivers a completion signal,
+		// which skews the counting.
 		var collector = make(chan interface{})
 
 		numParallel := 0