From fde1b139be35c655437a8df668dddf0367c3a67c Mon Sep 17 00:00:00 2001 From: Stefan Dombrowski Date: Mon, 6 Jun 2022 08:23:37 +0000 Subject: [PATCH] os: don't check for IsExist in MkdirAll example If a directory already exists, then MkdirAll returns nil. Therefore the check with IsExist is not necessary. Change-Id: Idf83c056f64bb56f49eb2b649af7827b759bcd7c GitHub-Last-Rev: 1f29873d0cd852642938cbd899549b9bf04301da GitHub-Pull-Request: golang/go#53242 Reviewed-on: https://go-review.googlesource.com/c/go/+/410434 Run-TryBot: Ian Lance Taylor Run-TryBot: Ian Lance Taylor Auto-Submit: Ian Lance Taylor Auto-Submit: Ian Lance Taylor Reviewed-by: Ian Lance Taylor TryBot-Result: Gopher Robot Reviewed-by: Matthew Dempsky Reviewed-by: Rob Pike Reviewed-by: Ian Lance Taylor --- src/os/example_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/os/example_test.go b/src/os/example_test.go index 53e3c5227b..5c7c6eac87 100644 --- a/src/os/example_test.go +++ b/src/os/example_test.go @@ -255,7 +255,7 @@ func ExampleMkdir() { func ExampleMkdirAll() { err := os.MkdirAll("test/subdir", 0750) - if err != nil && !os.IsExist(err) { + if err != nil { log.Fatal(err) } err = os.WriteFile("test/subdir/testfile.txt", []byte("Hello, Gophers!"), 0660) -- 2.48.1