Updated echo server documentation.

Updated echo server documentation.

R=jimbe@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/6e9295e103550ec97cf9c5a81438df358721fdaf

[Sean Klein, 2015-10-28 21:53:14]

smklein@chromium.org changed reviewers:
+ jimbe@chromium.org

[jimbe, 2015-10-30 20:28:30]

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md
File examples/echo/README.md (right):

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md#new...
examples/echo/README.md:7: For more general introduction to Mojo, refer to the
[Mojo
s/For more general introduction to Mojo/For a deeper dive into this code/

The issue is that the tutorial is really about writing code, not about learning
Mojo theory.

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md#new...
examples/echo/README.md:183: condition for multiple clients. If a new client
binds to the server before the
For the first sentence, how about:
The OneAtATimeServer demonstrates a pattern that should usually be avoided
because it contains a race condition for multiple clients.

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md#new...
examples/echo/README.md:185: error. Unless you know what you are doing, it is
advised to avoid creating a
s/know what you are doing/have a specific use case for this behavior/

[Sean Klein, 2015-10-30 22:31:54]

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md
File examples/echo/README.md (right):

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md#new...
examples/echo/README.md:7: For more general introduction to Mojo, refer to the
[Mojo
On 2015/10/30 20:28:29, jimbe wrote:
> s/For more general introduction to Mojo/For a deeper dive into this code/
> 
> The issue is that the tutorial is really about writing code, not about
learning
> Mojo theory.

Done.

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md#new...
examples/echo/README.md:183: condition for multiple clients. If a new client
binds to the server before the
On 2015/10/30 20:28:29, jimbe wrote:
> For the first sentence, how about:
> The OneAtATimeServer demonstrates a pattern that should usually be avoided
> because it contains a race condition for multiple clients.

Done.

https://codereview.chromium.org/1409993007/diff/1/examples/echo/README.md#new...
examples/echo/README.md:185: error. Unless you know what you are doing, it is
advised to avoid creating a
On 2015/10/30 20:28:29, jimbe wrote:
> s/know what you are doing/have a specific use case for this behavior/

Done.

[jimbe, 2015-11-03 17:27:37]

lgtm

[Sean Klein, 2015-11-03 18:43:42]

Committed patchset #2 (id:20001) manually as
6e9295e103550ec97cf9c5a81438df358721fdaf (presubmit successful).