From 7d562b8460a210ad9d0b65a44fbfb71d2ea81789 Mon Sep 17 00:00:00 2001
From: Dmitri Shuralyov <dmitshur@golang.org>
Date: Wed, 10 Sep 2025 15:08:27 -0400
Subject: [PATCH] syscall: actually remove unreachable code

CL 702415 claimed to remove unreachable code, but in reality merely hid
it from vet's unreachable pass. Since the unreachable code isn't serving
an active role in the test, do remove it to simplify code.

Change-Id: I5905b8b566e4ca013bdd1202d1492e3eae6a5ede
Reviewed-on: https://go-review.googlesource.com/c/go/+/702575
Reviewed-by: Ian Lance Taylor <iant@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Mark Freeman <markfreeman@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
---
 src/syscall/env_unix_test.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/syscall/env_unix_test.go b/src/syscall/env_unix_test.go
index b44b5b1e17..a005060609 100644
--- a/src/syscall/env_unix_test.go
+++ b/src/syscall/env_unix_test.go
@@ -96,10 +96,7 @@ func TestClearenv(t *testing.T) {
 			defer func() {
 				ret = recover()
 			}()
-			if true { // defeat vet's unreachable pass
-				panic(nil)
-			}
-			return "should not return"
+			panic(nil)
 		}
 
 		// Allow panic(nil).
-- 
2.52.0