From cf0351bdf57337f40d2400ec61b12de572f873b5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Mart=C3=AD?= Date: Fri, 11 Mar 2022 16:02:23 +0000 Subject: [PATCH] remove ErrExist check on -debugdir's RemoveAll This code initially used os.Stat, where it made sense to use os.IsNotExist to catch whether the directory didn't exist. However, we later transitioned to os.RemoveAll, which will never return neither ErrExist nor ErrNotExist: If the path does not exist, RemoveAll returns nil (no error). Simplify the code. --- main.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/main.go b/main.go index b5fb4e5..9c608d4 100644 --- a/main.go +++ b/main.go @@ -458,7 +458,7 @@ This command wraps "go %s". Below is its help: flagDebugDir = filepath.Join(wd, flagDebugDir) } - if err := os.RemoveAll(flagDebugDir); err == nil || errors.Is(err, fs.ErrExist) { + if err := os.RemoveAll(flagDebugDir); err == nil { err := os.MkdirAll(flagDebugDir, 0o755) if err != nil { return nil, err