LGTM
On Fri, May 9, 2014 at 2:56 PM, <iant@golang.org> wrote:
> Reviewers: golang-codereviews,
>
> Message:
> Hello golang-codereviews@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go
>
>
> Description:
> net/url: correct documentation of Values.Add
>
> Fixes issue 7816.
>
> Please review this at https://codereview.appspot.com/98160043/
>
> Affected files (+1, -1 lines):
> M src/pkg/net/url/url.go
>
>
> Index: src/pkg/net/url/url.go
> ===================================================================
> --- a/src/pkg/net/url/url.go
> +++ b/src/pkg/net/url/url.go
> @@ -502,7 +502,7 @@
> v[key] = []string{value}
> }
>
> -// Add adds the key to value. It appends to any existing
> +// Add adds the value to key. It appends to any existing
> // values associated with key.
> func (v Values) Add(key, value string) {
> v[key] = append(v[key], value)
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>
Issue 98160043: code review 98160043: net/url: correct documentation of Values.Add
(Closed)
Created 11 years ago by iant
Modified 11 years ago
Reviewers:
Base URL:
Comments: 0