Use more efficient ReadDir instead of Readdir

[goredo.git] / cleanup.go

diff --git a/cleanup.go b/cleanup.go
index bb9564cdc43285bd97472774693771bbd5feef04..d0b508d44201a7645717f77192b653cd5f4f30ef 100644 (file)
--- a/cleanup.go
+++ b/cleanup.go
@@ -1,6 +1,6 @@
 /*
-goredo -- redo implementation on pure Go
-Copyright (C) 2020 Sergey Matveev <stargrave@stargrave.org>
+goredo -- djb's redo implementation on pure Go
+Copyright (C) 2020-2023 Sergey Matveev <stargrave@stargrave.org>
 
 This program is free software: you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
@@ -18,10 +18,12 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 package main
 
 import (
+       "flag"
        "fmt"
        "io"
        "log"
        "os"
+       "path"
        "path/filepath"
        "strings"
 )
@@ -30,8 +32,18 @@ const (
        CleanupFull = "full"
        CleanupLog  = "log"
        CleanupTmp  = "tmp"
+       CleanupLock = "lock"
 )
 
+var DryRun *bool
+
+func init() {
+       if CmdName() != CmdNameRedoCleanup {
+               return
+       }
+       DryRun = flag.Bool("n", false, "do no delete files during cleanup, just show them")
+}
+
 func redoDirClean(root, what string) error {
        root, err := filepath.Abs(root)
        if err != nil {
@@ -41,8 +53,9 @@ func redoDirClean(root, what string) error {
        if err != nil {
                return err
        }
+       defer dir.Close()
        for {
-               fis, err := dir.Readdir(1 << 10)
+               entries, err := dir.ReadDir(1 << 10)
                if err != nil {
                        if err == io.EOF {
                                break
@@ -50,21 +63,35 @@ func redoDirClean(root, what string) error {
                        return err
                }
                var pth string
-               for _, fi := range fis {
-                       pth = cwdMustRel(root, fi.Name())
+               for _, entry := range entries {
+                       pth = cwdMustRel(root, entry.Name())
                        switch what {
                        case CleanupLog:
-                               if strings.HasSuffix(fi.Name(), LogSuffix) {
+                               if strings.HasSuffix(entry.Name(), LogSuffix) ||
+                                       strings.HasSuffix(entry.Name(), LogRecSuffix) {
                                        fmt.Println(pth)
-                                       if err = os.Remove(pth); err != nil {
-                                               return err
+                                       if !*DryRun {
+                                               if err = os.Remove(pth); err != nil {
+                                                       return err
+                                               }
+                                       }
+                               }
+                       case CleanupLock:
+                               if strings.HasSuffix(entry.Name(), LockSuffix) {
+                                       fmt.Println(pth)
+                                       if !*DryRun {
+                                               if err = os.Remove(pth); err != nil {
+                                                       return err
+                                               }
                                        }
                                }
                        case CleanupTmp:
-                               if strings.HasPrefix(fi.Name(), TmpPrefix) {
+                               if strings.HasPrefix(entry.Name(), TmpPrefix) {
                                        fmt.Println(pth)
-                                       if err = os.Remove(pth); err != nil {
-                                               return err
+                                       if !*DryRun {
+                                               if err = os.Remove(pth); err != nil {
+                                                       return err
+                                               }
                                        }
                                }
                        default:
@@ -72,7 +99,7 @@ func redoDirClean(root, what string) error {
                        }
                }
        }
-       return dir.Close()
+       return nil
 }
 
 func cleanupWalker(root, what string) error {
@@ -86,24 +113,32 @@ func cleanupWalker(root, what string) error {
        }
        defer dir.Close()
        for {
-               fis, err := dir.Readdir(1 << 10)
+               entries, err := dir.ReadDir(1 << 10)
                if err != nil {
                        if err == io.EOF {
                                break
                        }
                        return err
                }
-               var pth string
-               for _, fi := range fis {
-                       pth = cwdMustRel(root, fi.Name())
-                       if fi.IsDir() {
-                               if fi.Name() == RedoDir {
+               for _, entry := range entries {
+                       pth := path.Join(root, entry.Name())
+                       pthRel := cwdMustRel(root, entry.Name())
+                       if entry.IsDir() {
+                               if entry.Name() == RedoDir {
                                        if what == CleanupFull {
-                                               fmt.Println(pth)
-                                               err = os.RemoveAll(pth)
+                                               fmt.Println(pthRel)
+                                               if !*DryRun {
+                                                       err = os.RemoveAll(pth)
+                                               }
                                        } else {
                                                err = redoDirClean(pth, what)
                                        }
+                               } else if (what == CleanupTmp || what == CleanupFull) &&
+                                       strings.HasPrefix(entry.Name(), TmpPrefix) {
+                                       fmt.Println(pthRel)
+                                       if !*DryRun {
+                                               err = os.RemoveAll(pth)
+                                       }
                                } else {
                                        err = cleanupWalker(pth, what)
                                }
@@ -113,13 +148,15 @@ func cleanupWalker(root, what string) error {
                                continue
                        }
                        if (what == CleanupTmp || what == CleanupFull) &&
-                               strings.HasPrefix(fi.Name(), TmpPrefix) {
-                               fmt.Println(pth)
-                               if err = os.Remove(pth); err != nil {
-                                       return err
+                               strings.HasPrefix(entry.Name(), TmpPrefix) {
+                               fmt.Println(pthRel)
+                               if !*DryRun {
+                                       if err = os.Remove(pth); err != nil {
+                                               return err
+                                       }
                                }
                        }
                }
        }
-       return dir.Close()
+       return nil
 }