Merge pull request #269 from durin42/small-fixes
Small fixes I noticed while perusing the code.
diff --git a/config_test.go b/config_test.go
index 8c2decc..fea8d8a 100644
--- a/config_test.go
+++ b/config_test.go
@@ -18,10 +18,22 @@
return nil, err
}
- c.SetString("foo.bar", "baz")
- c.SetBool("foo.bool", true)
- c.SetInt32("foo.int32", 32)
- c.SetInt64("foo.int64", 64)
+ err = c.SetString("foo.bar", "baz")
+ if err != nil {
+ return nil, err
+ }
+ err = c.SetBool("foo.bool", true)
+ if err != nil {
+ return nil, err
+ }
+ err = c.SetInt32("foo.int32", 32)
+ if err != nil {
+ return nil, err
+ }
+ err = c.SetInt64("foo.int64", 64)
+ if err != nil {
+ return nil, err
+ }
return c, err
}
@@ -86,6 +98,7 @@
if err != nil {
t.Errorf("Setup error: '%v'. Expected none\n", err)
+ return
}
defer c.Free()
diff --git a/odb.go b/odb.go
index be0870e..b15851f 100644
--- a/odb.go
+++ b/odb.go
@@ -11,7 +11,6 @@
"reflect"
"runtime"
"unsafe"
- "fmt"
)
type Odb struct {
@@ -107,9 +106,7 @@
}
err := data.callback(newOidFromC(id))
- fmt.Println("err %v", err)
if err != nil {
- fmt.Println("returning EUSER")
data.err = err
return C.GIT_EUSER
}
@@ -130,7 +127,6 @@
defer pointerHandles.Untrack(handle)
ret := C._go_git_odb_foreach(v.ptr, handle)
- fmt.Println("ret %v", ret);
if ret == C.GIT_EUSER {
return data.err
} else if ret < 0 {