From 45ac03181f61477d28e031725c68f938cb379554 Mon Sep 17 00:00:00 2001
From: "Santiago M. Mola" <santi@mola.io>
Date: Wed, 26 Apr 2017 13:26:51 +0200
Subject: fix format string issues as found by go vet

---
 plumbing/object/change_adaptor.go | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'plumbing/object')

diff --git a/plumbing/object/change_adaptor.go b/plumbing/object/change_adaptor.go
index cf8de6c..49b6545 100644
--- a/plumbing/object/change_adaptor.go
+++ b/plumbing/object/change_adaptor.go
@@ -1,6 +1,7 @@
 package object
 
 import (
+	"errors"
 	"fmt"
 
 	"gopkg.in/src-d/go-git.v4/utils/merkletrie"
@@ -15,11 +16,11 @@ func newChange(c merkletrie.Change) (*Change, error) {
 
 	var err error
 	if ret.From, err = newChangeEntry(c.From); err != nil {
-		return nil, fmt.Errorf("From field: ", err)
+		return nil, fmt.Errorf("From field: %s", err)
 	}
 
 	if ret.To, err = newChangeEntry(c.To); err != nil {
-		return nil, fmt.Errorf("To field: ", err)
+		return nil, fmt.Errorf("To field: %s", err)
 	}
 
 	return ret, nil
@@ -32,7 +33,7 @@ func newChangeEntry(p noder.Path) (ChangeEntry, error) {
 
 	asTreeNoder, ok := p.Last().(*treeNoder)
 	if !ok {
-		return ChangeEntry{}, fmt.Errorf("cannot transform non-TreeNoders")
+		return ChangeEntry{}, errors.New("cannot transform non-TreeNoders")
 	}
 
 	return ChangeEntry{
@@ -52,7 +53,7 @@ func newChanges(src merkletrie.Changes) (Changes, error) {
 	for i, e := range src {
 		ret[i], err = newChange(e)
 		if err != nil {
-			return nil, fmt.Errorf("change #%d: %s", err)
+			return nil, fmt.Errorf("change #%d: %s", i, err)
 		}
 	}
 
-- 
cgit 


From 4b5b96270ad44333afd8fe8ac17d320459ad1cf1 Mon Sep 17 00:00:00 2001
From: "Santiago M. Mola" <santi@mola.io>
Date: Wed, 26 Apr 2017 13:35:20 +0200
Subject: object: avoid unreachable code

---
 plumbing/object/change_test.go | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

(limited to 'plumbing/object')

diff --git a/plumbing/object/change_test.go b/plumbing/object/change_test.go
index 005ceb0..bfd4613 100644
--- a/plumbing/object/change_test.go
+++ b/plumbing/object/change_test.go
@@ -236,10 +236,7 @@ func (s *ChangeSuite) TestNoFileFilemodes(c *C) {
 		c.Assert(err, IsNil)
 		for _, change := range changes {
 			_, _, err := change.Files()
-			if err != nil {
-				panic(err)
-				c.Assert(err, IsNil)
-			}
+			c.Assert(err, IsNil)
 		}
 
 		prev = commit
-- 
cgit