Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1339)

Unified Diff: examples/echo/README.md

Issue 1409993007: Updated echo server documentation. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: examples/echo/README.md
diff --git a/examples/echo/README.md b/examples/echo/README.md
index b2d4b55e57ba85f6f7232d2aa94730c6afe13278..04c5c007ccf1530e9dccc65a29437a781f8f154b 100644
--- a/examples/echo/README.md
+++ b/examples/echo/README.md
@@ -4,6 +4,9 @@ Example Echo Client & Server
This echo client/server demonstrate how to create and use a mojom interface,
as well as demonstrating one way to communicate between mojo applications.
+For a deeper dive into this code, refer to the [Mojo
+Tutorial](https://docs.google.com/document/d/1mufrtxTk8w9qa3jcnlgqsYkWlyhwEpc7aWNaSOks7ug).
+
## Running the Echo Client & Server
```
@@ -175,3 +178,9 @@ This server creates an `EchoImpl` object, like the `SingletonServer`, but uses a
single `Binding`, rather than a `BindingSet`. This means that when a new client
connects to the OneAtATimeServer, the previous binding is closed, and a new
binding is made between the new client and the interface implementation.
+
+The OneAtATimeServer demonstrates a pattern that should be avoided because it
+contains a race condition for multiple clients. If a new client binds to the
+server before the first client managed to call EchoString, the first client's
+call would cause an error. Unless you have a specific use case for this
+behavior, it is advised to avoid creating a server like this.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698