From 99d7f66477aa09915821659087da8b91e593f4eb Mon Sep 17 00:00:00 2001
From: Jesse Ezell <jesse.ezell@neudesic.com>
Date: Thu, 20 Mar 2014 02:06:56 -0700
Subject: [PATCH 1/2] add remote list

---
 remote.go      | 12 ++++++++++++
 remote_test.go | 20 ++++++++++++++++++++
 2 files changed, 32 insertions(+)

diff --git a/remote.go b/remote.go
index d556f99..d675ab8 100644
--- a/remote.go
+++ b/remote.go
@@ -132,6 +132,18 @@ func (r *Remote) Free() {
 	C.git_remote_free(r.ptr)
 }
 
+func (repo *Repository) ListRemotes() ([]string, error) {
+	var r C.git_strarray
+	ecode := C.git_remote_list(&r, repo.ptr)
+	if ecode < 0 {
+		return make([]string, 0), MakeGitError(ecode)
+	}
+	defer C.git_strarray_free(&r)
+
+	remotes := makeStringsFromCStrings(r.strings, int(r.count))
+	return remotes, nil
+}
+
 func (repo *Repository) CreateRemote(name string, url string) (*Remote, error) {
 	remote := &Remote{}
 
diff --git a/remote_test.go b/remote_test.go
index 04b3a57..90e24ae 100644
--- a/remote_test.go
+++ b/remote_test.go
@@ -8,6 +8,7 @@ import (
 func TestRefspecs(t *testing.T) {
 	repo := createTestRepo(t)
 	defer os.RemoveAll(repo.Workdir())
+	defer repo.Free()
 
 	remote, err := repo.CreateRemoteInMemory("refs/heads/*:refs/heads/*", "git://foo/bar")
 	checkFatal(t, err)
@@ -25,3 +26,22 @@ func TestRefspecs(t *testing.T) {
 
 	compareStringList(t, expected, actual)
 }
+
+func TestListRemotes(t *testing.T) {
+	repo := createTestRepo(t)
+	defer os.RemoveAll(repo.Workdir())
+	defer repo.Free()
+
+	_, err := repo.CreateRemote("test", "git://foo/bar")
+
+	checkFatal(t, err)
+
+	expected := []string{
+		"test",
+	}
+
+	actual, err := repo.ListRemotes()
+	checkFatal(t, err)
+
+	compareStringList(t, expected, actual)
+}

From 5d8db7f9362a314cf56747cf23605aec8640e92e Mon Sep 17 00:00:00 2001
From: Jesse Ezell <jesse.ezell@neudesic.com>
Date: Thu, 20 Mar 2014 20:24:29 -0700
Subject: [PATCH 2/2] return nil instead of empty array on error

---
 remote.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/remote.go b/remote.go
index d675ab8..66097b8 100644
--- a/remote.go
+++ b/remote.go
@@ -136,7 +136,7 @@ func (repo *Repository) ListRemotes() ([]string, error) {
 	var r C.git_strarray
 	ecode := C.git_remote_list(&r, repo.ptr)
 	if ecode < 0 {
-		return make([]string, 0), MakeGitError(ecode)
+		return nil, MakeGitError(ecode)
 	}
 	defer C.git_strarray_free(&r)