Packbuilder: compilation fixes

Don't name the return values, as they conflict with the names we want
inside and the types don't match what we want to have inside. We need
them to be two-way channels in the function, and then pass
unidirectional references to the different functions.

2 files changed, 1 insertions, 2 deletions

diff --git a/git.go b/git.go
index d8cacc4..6f81293 100644
--- a/git.go
+++ b/git.go
@@ -11,7 +11,6 @@ import (
 	"errors"
 	"unsafe"
 	"strings"
-	"fmt"
 )
 
 const (
diff --git a/packbuilder.go b/packbuilder.go
index 292604d..bcb61a4 100644
--- a/packbuilder.go
+++ b/packbuilder.go
@@ -121,7 +121,7 @@ func (pb *Packbuilder) forEachWrap(data *packbuilderCbData) {
 // you want to stop the pack-building process (e.g. there's an error
 // writing to the output), close or write a value into the "stop"
 // channel.
-func (pb *Packbuilder) ForEach() (data <-chan []byte, stop chan<- bool) {
+func (pb *Packbuilder) ForEach() (<-chan []byte, chan<- bool) {
 	ch := make(chan []byte)
 	stop := make(chan bool)
 	data := packbuilderCbData{ch, stop}