code review 4168043: json: correct Marshal documentation

json: correct Marshal documentation

Fixes issue 1488.

[adg, 2011-02-10 15:53:40]

Hello r (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[rsc, 2011-02-10 15:58:57]

LGTM

[rog, 2011-02-10 16:07:13]

don't most JSON fields use lower case, at least for the first letter?
maybe that might be a better default, otherwise everyone will need
to tag most fields.

[rsc, 2011-02-10 16:08:54]

On Thu, Feb 10, 2011 at 11:07,  <rogpeppe@gmail.com> wrote:
> don't most JSON fields use lower case, at least for the first letter?
> maybe that might be a better default, otherwise everyone will need
> to tag most fields.

Some services use upper case, others do not.
Easier to have no mapping and override.

Needing to say

    Foo int "Foo"

seems very wrong.

Russ

[rog, 2011-02-10 16:11:57]

LGTM

On 10 February 2011 16:08, Russ Cox <rsc@golang.org> wrote:
> On Thu, Feb 10, 2011 at 11:07,  <rogpeppe@gmail.com> wrote:
>> don't most JSON fields use lower case, at least for the first letter?
>> maybe that might be a better default, otherwise everyone will need
>> to tag most fields.
>
> Some services use upper case, others do not.
> Easier to have no mapping and override.
>
> Needing to say
>
>    Foo int "Foo"
>
> seems very wrong.
>
> Russ
>

[adg, 2011-02-10 18:48:16]

*** Submitted as http://code.google.com/p/go/source/detail?r=9dda1be7ffa7 ***

json: correct Marshal documentation

Fixes issue 1488.

R=r, rsc, rog
CC=golang-dev
http://codereview.appspot.com/4168043