code review 5846044: doc: add JSON and Go article

doc: add JSON and Go article

Originally published on The Go Programming Language Blog, January 25, 2011.

http://blog.golang.org/2011/01/json-and-go.html

[fss, 2012-03-16 05:54:33]

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

I'd like you to review this change to
https://code.google.com/p/go

[fss, 2012-03-19 22:59:31]

ping?

[adg, 2012-03-20 02:37:34]

https://codereview.appspot.com/5846044/diff/4002/doc/articles/json_and_go.html
File doc/articles/json_and_go.html (right):

https://codereview.appspot.com/5846044/diff/4002/doc/articles/json_and_go.htm...
doc/articles/json_and_go.html:29: {{code "/src/pkg/encoding/json/encode.go"
`/func Marshal/`}}
Just write this out explicitly, so we don't get the {
Same for Unmarshal, NewEncoder, NewDecoder, etc

https://codereview.appspot.com/5846044/diff/4002/doc/progs/json1.go
File doc/progs/json1.go (right):

https://codereview.appspot.com/5846044/diff/4002/doc/progs/json1.go#newcode29
doc/progs/json1.go:29: reflect.DeepEqual(b, []byte(`{"Name": "Alice", "Body":
"Hello", "Time": 1294706395881547000}`))
why do this if you don't check it?

In fact, all of these should be checked

[fss, 2012-03-21 17:08:50]

ptal

Now ./run uses testit to validate the output of json[1-4].

json5 is a little tricky, since it is an infinite loop. If you have any
recommendation, please let me know.

Sorry for the delay.

https://codereview.appspot.com/5846044/diff/4002/doc/articles/json_and_go.html
File doc/articles/json_and_go.html (right):

https://codereview.appspot.com/5846044/diff/4002/doc/articles/json_and_go.htm...
doc/articles/json_and_go.html:29: {{code "/src/pkg/encoding/json/encode.go"
`/func Marshal/`}}
On 2012/03/20 02:37:35, adg wrote:
> Just write this out explicitly, so we don't get the {
> Same for Unmarshal, NewEncoder, NewDecoder, etc

Done.

https://codereview.appspot.com/5846044/diff/4002/doc/progs/json1.go
File doc/progs/json1.go (right):

https://codereview.appspot.com/5846044/diff/4002/doc/progs/json1.go#newcode29
doc/progs/json1.go:29: reflect.DeepEqual(b, []byte(`{"Name": "Alice", "Body":
"Hello", "Time": 1294706395881547000}`))
On 2012/03/20 02:37:35, adg wrote:
> why do this if you don't check it?
> 
> In fact, all of these should be checked

Done.

[adg, 2012-03-22 07:25:08]

LGTM

[adg, 2012-03-22 07:25:47]

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

doc: add JSON and Go article

Originally published on The Go Programming Language Blog, January 25, 2011.

http://blog.golang.org/2011/01/json-and-go.html

R=adg
CC=golang-dev
http://codereview.appspot.com/5846044

Committer: Andrew Gerrand <adg@golang.org>