code review 2833041: netchan: fix locking bug.

netchan: fix locking bug.
There's no need to hold the client mutex when calling encode, since encode itself
uses a mutex to make the writes atomic.  However, we need to keep
the messages ordered, so add a mutex for that purpose alone.
Fixes issue 1244.

[r, 2010-11-02 17:07:11]

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

I'd like you to review this change.

[r, 2010-11-02 17:56:46]

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

Please take another look.

[rsc1, 2010-11-02 17:57:47]

LGTM

[r, 2010-11-02 17:58:30]

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

netchan: fix locking bug.
There's no need to hold the client mutex when calling encode, since encode
itself
uses a mutex to make the writes atomic.  However, we need to keep
the messages ordered, so add a mutex for that purpose alone.
Fixes issue 1244.

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