diff options
| author | Felipe Leopoldo Sologuren Gutiérrez <fsologureng@users.noreply.github.com> | 2023-01-16 13:21:44 -0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-01-16 16:21:44 +0000 |
| commit | 04c97aa36473bc0070a2fe46e86dc645dc75ee85 (patch) | |
| tree | 409f98cebeaac4b9f8b2482574963c78897710bc /modules/repository/generate.go | |
| parent | da274380a7be72e0153a448611278b402e20ad7e (diff) | |
| download | gitea-04c97aa36473bc0070a2fe46e86dc645dc75ee85.tar.gz gitea-04c97aa36473bc0070a2fe46e86dc645dc75ee85.zip | |
Change use of Walk to WalkDir to improve disk performance (#22462)
As suggest by Go developers, use `filepath.WalkDir` instead of
`filepath.Walk` because [*Walk is less efficient than WalkDir,
introduced in Go 1.16, which avoids calling `os.Lstat` on every file or
directory visited](https://pkg.go.dev/path/filepath#Walk).
This proposition address that, in a similar way as
https://github.com/go-gitea/gitea/pull/22392 did.
Co-authored-by: zeripath <art27@cantab.net>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Diffstat (limited to 'modules/repository/generate.go')
| -rw-r--r-- | modules/repository/generate.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/modules/repository/generate.go b/modules/repository/generate.go index d72934729c..b6a1d7b43e 100644 --- a/modules/repository/generate.go +++ b/modules/repository/generate.go @@ -173,12 +173,12 @@ func generateRepoCommit(ctx context.Context, repo, templateRepo, generateRepo *r // Avoid walking tree if there are no globs if len(gt.Globs()) > 0 { tmpDirSlash := strings.TrimSuffix(filepath.ToSlash(tmpDir), "/") + "/" - if err := filepath.Walk(tmpDirSlash, func(path string, info os.FileInfo, walkErr error) error { + if err := filepath.WalkDir(tmpDirSlash, func(path string, d os.DirEntry, walkErr error) error { if walkErr != nil { return walkErr } - if info.IsDir() { + if d.IsDir() { return nil } |