better handling of empty values in slices/arrays

if the slice or array has no values, skip it entirely.  This fixes
an issue where a struct field with a custom delimiter was included in
the returned url.Values with an empty string rather than being omitted
entirely (as is the case when there is no custom delimiter)

fixes #57

Author: willnorris

query/encode.go

@@ -209,6 +209,11 @@ func reflectValue(values url.Values, val reflect.Value, scope string) error {
 		}
 
 		if sv.Kind() == reflect.Slice || sv.Kind() == reflect.Array {
+			if sv.Len() == 0 {
+				// skip if slice or array is empty
+				continue
+			}
+
 			var del string
 			if opts.Contains("comma") {
 				del = ","

query/encode_test.go

@@ -143,10 +143,30 @@ func TestValues_Slices(t *testing.T) {
 			struct{ V []string }{},
 			url.Values{},
 		},
+		{
+			struct{ V []string }{[]string{}},
+			url.Values{},
+		},
+		{
+			struct{ V []string }{[]string{""}},
+			url.Values{"V": {""}},
+		},
 		{
 			struct{ V []string }{[]string{"a", "b"}},
 			url.Values{"V": {"a", "b"}},
 		},
+		{
+			struct {
+				V []string `url:",comma"`
+			}{[]string{}},
+			url.Values{},
+		},
+		{
+			struct {
+				V []string `url:",comma"`
+			}{[]string{""}},
+			url.Values{"V": {""}},
+		},
 		{
 			struct {
 				V []string `url:",comma"`