*: simplication

 - no unnecessary err/bool check, uses them directly

Signed-off-by: ferhat elmas <elmas.ferhat@gmail.com>

3 files changed, 4 insertions, 11 deletions

diff --git a/object_walker.go b/object_walker.go
index 8bae1fa..4cbbcca 100644
--- a/object_walker.go
+++ b/object_walker.go
@@ -36,10 +36,7 @@ func (p *objectWalker) walkAllRefs() error {
 		}
 		return p.walkObjectTree(ref.Hash())
 	})
-	if err != nil {
-		return err
-	}
-	return nil
+	return err
 }
 
 func (p *objectWalker) isSeen(hash plumbing.Hash) bool {
diff --git a/storage/filesystem/internal/dotgit/dotgit.go b/storage/filesystem/internal/dotgit/dotgit.go
index ebc6bad..d644810 100644
--- a/storage/filesystem/internal/dotgit/dotgit.go
+++ b/storage/filesystem/internal/dotgit/dotgit.go
@@ -212,11 +212,7 @@ func (d *DotGit) DeleteOldObjectPackAndIndex(hash plumbing.Hash, t time.Time) er
 	if err != nil {
 		return err
 	}
-	err = d.fs.Remove(d.objectPackPath(hash, `idx`))
-	if err != nil {
-		return err
-	}
-	return nil
+	return d.fs.Remove(d.objectPackPath(hash, `idx`))
 }
 
 // NewObject return a writer for a new object file.
diff --git a/worktree_test.go b/worktree_test.go
index 36e3a08..ee9bf2f 100644
--- a/worktree_test.go
+++ b/worktree_test.go
@@ -1481,7 +1481,7 @@ func (s *WorktreeSuite) TestGrep(c *C) {
 					break
 				}
 			}
-			if found != true {
+			if !found {
 				c.Errorf("unexpected grep results for %q, expected result to contain: %v", tc.name, wantResult)
 			}
 		}
@@ -1496,7 +1496,7 @@ func (s *WorktreeSuite) TestGrep(c *C) {
 					break
 				}
 			}
-			if found != false {
+			if found {
 				c.Errorf("unexpected grep results for %q, expected result to NOT contain: %v", tc.name, dontWantResult)
 			}
 		}