From b5aefe07e5afe2fd5d49c6a4219cc826b3e4e34e Mon Sep 17 00:00:00 2001 From: Tobias Klauser Date: Tue, 14 Oct 2025 11:40:26 +0200 Subject: all: remove unnecessary loop variable copies in tests Copying the loop variable is no longer necessary since Go 1.22. Change-Id: Iebb21dac44a20ec200567f1d786f105a4ee4999d Reviewed-on: https://go-review.googlesource.com/c/go/+/711640 Reviewed-by: Florian Lehner Auto-Submit: Damien Neil Reviewed-by: Dmitri Shuralyov Reviewed-by: Damien Neil Auto-Submit: Tobias Klauser LUCI-TryBot-Result: Go LUCI --- src/internal/trace/reader_test.go | 1 - src/internal/trace/tracev1_test.go | 1 - 2 files changed, 2 deletions(-) (limited to 'src/internal/trace') diff --git a/src/internal/trace/reader_test.go b/src/internal/trace/reader_test.go index 39ae77471e..e0733b827d 100644 --- a/src/internal/trace/reader_test.go +++ b/src/internal/trace/reader_test.go @@ -33,7 +33,6 @@ func TestReaderGolden(t *testing.T) { t.Fatalf("failed to glob for tests: %v", err) } for _, testPath := range matches { - testPath := testPath testName, err := filepath.Rel("./testdata", testPath) if err != nil { t.Fatalf("failed to relativize testdata path: %v", err) diff --git a/src/internal/trace/tracev1_test.go b/src/internal/trace/tracev1_test.go index 42a8371887..355a6ff529 100644 --- a/src/internal/trace/tracev1_test.go +++ b/src/internal/trace/tracev1_test.go @@ -21,7 +21,6 @@ func TestTraceV1(t *testing.T) { } var testedUserRegions bool for _, p := range traces { - p := p testName, err := filepath.Rel("./internal/tracev1/testdata", p) if err != nil { t.Fatalf("failed to relativize testdata path: %s", err) -- cgit v1.3