Browse Source

Merge pull request #1218 from unihorn/147

flags/urls: assign urls instead of append
Yicheng Qin 11 years ago
parent
commit
c0c0b08ff2
1 changed files with 2 additions and 2 deletions
  1. 2 2
      pkg/flags/urls.go

+ 2 - 2
pkg/flags/urls.go

@@ -19,7 +19,7 @@ func (us *URLs) Set(s string) error {
 	if len(all) == 0 {
 	if len(all) == 0 {
 		return errors.New("no valid URLs given")
 		return errors.New("no valid URLs given")
 	}
 	}
-	for _, in := range strs {
+	for i, in := range strs {
 		in = strings.TrimSpace(in)
 		in = strings.TrimSpace(in)
 		u, err := url.Parse(in)
 		u, err := url.Parse(in)
 		if err != nil {
 		if err != nil {
@@ -31,7 +31,7 @@ func (us *URLs) Set(s string) error {
 		if u.Path != "" {
 		if u.Path != "" {
 			return fmt.Errorf("URL must not contain a path: %s", s)
 			return fmt.Errorf("URL must not contain a path: %s", s)
 		}
 		}
-		all = append(all, *u)
+		all[i] = *u
 	}
 	}
 	*us = all
 	*us = all
 	return nil
 	return nil