mirror of
https://github.com/golang/go.git
synced 2025-02-06 10:03:36 +00:00
testing: use strings.SplitSeq and bytes.SplitSeq
To simplify the code. This is a follow-up for the CL 646216. Change-Id: Ib09d1074a783482fb293527e9f1abeb3c02137c3 GitHub-Last-Rev: 2e7a6ad40cc22ea855e4d703ff39db9cc2c8a58e GitHub-Pull-Request: golang/go#71568 Reviewed-on: https://go-review.googlesource.com/c/go/+/646755 Reviewed-by: Jorropo <jorropo.pgm@gmail.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Salah (Globlost) <globlost@gmail.com> Reviewed-by: Damien Neil <dneil@google.com> Auto-Submit: Damien Neil <dneil@google.com> Reviewed-by: Alan Donovan <adonovan@google.com>
This commit is contained in:
parent
51bf2cf7cf
commit
4cc7705e56
@ -23,7 +23,7 @@ func Example_parsing() {
|
||||
|
||||
results := func() []map[string]any {
|
||||
var ms []map[string]any
|
||||
for _, line := range bytes.Split(buf.Bytes(), []byte{'\n'}) {
|
||||
for line := range bytes.SplitSeq(buf.Bytes(), []byte{'\n'}) {
|
||||
if len(line) == 0 {
|
||||
continue
|
||||
}
|
||||
|
@ -2501,7 +2501,7 @@ func (m *M) stopAlarm() {
|
||||
}
|
||||
|
||||
func parseCpuList() {
|
||||
for _, val := range strings.Split(*cpuListStr, ",") {
|
||||
for val := range strings.SplitSeq(*cpuListStr, ",") {
|
||||
val = strings.TrimSpace(val)
|
||||
if val == "" {
|
||||
continue
|
||||
|
@ -894,7 +894,7 @@ func TestRunningTestsInCleanup(t *testing.T) {
|
||||
|
||||
func parseRunningTests(out []byte) (runningTests []string, ok bool) {
|
||||
inRunningTests := false
|
||||
for _, line := range strings.Split(string(out), "\n") {
|
||||
for line := range strings.SplitSeq(string(out), "\n") {
|
||||
if inRunningTests {
|
||||
// Package testing adds one tab, the panic printer adds another.
|
||||
if trimmed, ok := strings.CutPrefix(line, "\t\t"); ok {
|
||||
|
Loading…
x
Reference in New Issue
Block a user