From 0d46e1b419057da6e02f9e1b956145c3bc652892 Mon Sep 17 00:00:00 2001
From: 178inaba <178inaba@users.noreply.github.com>
Date: Thu, 4 May 2017 16:38:13 +0900
Subject: [PATCH] Fix to return nil if parse empty string

---
 main.go      | 10 ++++++++--
 main_test.go |  7 +++++++
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/main.go b/main.go
index ce3b24a..944637e 100644
--- a/main.go
+++ b/main.go
@@ -63,6 +63,10 @@ func (l Links) FilterByRel(r string) Links {
 // String returns the string representation of multiple Links
 // for use in HTTP responses etc
 func (l Links) String() string {
+	if l == nil {
+		return fmt.Sprint(nil)
+	}
+
 	var strs []string
 	for _, link := range l {
 		strs = append(strs, link.String())
@@ -74,7 +78,7 @@ func (l Links) String() string {
 //   <url>; rel="foo", <url>; rel="bar"; wat="dis"
 // returning a slice of Link structs
 func Parse(raw string) Links {
-	links := make(Links, 0)
+	var links Links
 
 	// One chunk: <url>; rel="foo"
 	for _, chunk := range strings.Split(raw, ",") {
@@ -110,7 +114,9 @@ func Parse(raw string) Links {
 
 		}
 
-		links = append(links, link)
+		if link.URL != "" && link.Rel != "" {
+			links = append(links, link)
+		}
 	}
 
 	return links
diff --git a/main_test.go b/main_test.go
index 6c8a4a0..d9eef51 100644
--- a/main_test.go
+++ b/main_test.go
@@ -38,6 +38,13 @@ func TestSimple(t *testing.T) {
 
 }
 
+func TestEmpty(t *testing.T) {
+	links := Parse("")
+	if links != nil {
+		t.Errorf("Return value should be nil, but was %s", len(links))
+	}
+}
+
 func TestLinkMethods(t *testing.T) {
 	header := "<https://api.github.com/user/9287/repos?page=1&per_page=100>; rel=\"prev\"; pet=\"cat\""
 	links := Parse(header)