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

Unified Diff: runtime/vm/service/protocol.md

Issue 514833003: Add a new Sentinel type for sentinel responses. (They are not Null). (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: gen js, fix tests Created 6 years, 4 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 | « runtime/vm/service.cc ('k') | runtime/vm/service_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/service/protocol.md
diff --git a/runtime/vm/service/protocol.md b/runtime/vm/service/protocol.md
index 6500ebdaded2951e78536d7e6df271dd5945f49c..d2e70ff1f9915b2b0b5fb49d0ddd8077845404ea 100644
--- a/runtime/vm/service/protocol.md
+++ b/runtime/vm/service/protocol.md
@@ -429,6 +429,17 @@ TODO: Split Null from the other Sentinel types.
| 1 + (2 * k) | int | kth token position
| 2 + (2 * k) | int | kth column number
+### <a name="Sentinel"></a>Sentinel
+
+TODO: Enumerate known Sentinels
+TODO: Should this even have an id? Maybe a *kind* instead.
+
+| keys | values | comments
+| --- | --- | ---
+| type | "Sentinel" |
+| id | String | |
+| valueAsString | String |
+
### <a name="VM"></a>VM
| keys | values | comments
« no previous file with comments | « runtime/vm/service.cc ('k') | runtime/vm/service_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698