From ee62878e6b260645a7a3e1c377ecbea275a50c48 Mon Sep 17 00:00:00 2001 From: Stefan Haller Date: Wed, 2 Jul 2025 15:47:48 +0200 Subject: [PATCH] Cleanup: remove pointless error handling ForEachFile can't possibly return an error if the passed function doesn't return one. --- pkg/gui/controllers/commits_files_controller.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/pkg/gui/controllers/commits_files_controller.go b/pkg/gui/controllers/commits_files_controller.go index 8beeb6ff2..2957c8f00 100644 --- a/pkg/gui/controllers/commits_files_controller.go +++ b/pkg/gui/controllers/commits_files_controller.go @@ -319,13 +319,10 @@ func (self *CommitFilesController) discard(selectedNodes []*filetree.CommitFileN } for _, node := range selectedNodes { - err := node.ForEachFile(func(file *models.CommitFile) error { + _ = node.ForEachFile(func(file *models.CommitFile) error { filePaths = append(filePaths, file.GetPath()) return nil }) - if err != nil { - return err - } } err := self.c.Git().Rebase.DiscardOldFileChanges(self.c.Model().Commits, self.c.Contexts().LocalCommits.GetSelectedLineIdx(), filePaths)