encoding/json: simplify (*decodeState).indirect
encoding/json: simplify (*decodeState).indirect

Removes an incorrect code comment and some superfluous variables.

The comment I removed says that struct fields which implement
Unmarshaler must be pointers, even if they're in an addressable
struct. That's not the case, and there's already a test in decode_test.go
that demonstrates as much.

Encoding/json has quite a few assignments of reflect.Values to extra
variables – things like "iv := v" when there's no need to make a copy. I
think these are left over from a previous version of the reflect API. If they
aren't wanted, I wouldn't mind going through the package and getting
rid of the rest of them.

R=rsc
CC=golang-dev
http://codereview.appspot.com/6318047

Committer: Russ Cox <5ad239cb8a44f659eaaee0aa1ea5b94947abe557@golang.org>

gnanderson/go,pombredanne/go-deleteme,dsriram/golang,gnanderson/go,dsriram/golang,dsriram/golang,pombredanne/go-deleteme,gnanderson/go,dsriram/golang,tv42/old-go,dsriram/golang,gnanderson/go,pombredanne/go-deleteme,tv42/old-go,apparentlymart/golang,apparentlymart/golang,gnanderson/go,pombredanne/go-deleteme,apparentlymart/golang,dsriram/golang,apparentlymart/golang,tv42/old-go,pombredanne/go-deleteme,pombredanne/go-deleteme,tv42/old-go,gnanderson/go,gnanderson/go,pombredanne/go-deleteme,apparentlymart/golang,tv42/old-go,dsriram/golang,tv42/old-go,apparentlymart/golang,apparentlymart/golang,tv42/old-go