Uh oh!
There was an error while loading. Please reload this page.
Document JsonReader token values - #587
Conversation
Refs #247 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the XML documentation for Android.Util.JsonReader by replacing “To be added.” placeholders with concrete <value>/<returns> descriptions, aligning the docs more closely with the upstream Android reference documentation.
Changes:
- Documented
HasNextproperty value semantics (true/falsemeaning). - Added concrete
<returns>documentation forNextBoolean,NextDouble,NextInt,NextLong,NextName,NextString, andPeek.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Uh oh!
There was an error while loading. Please reload this page.
Refs #247 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
jonathanpeppers
commented
Aug 16, 2026
@dalexsoto review |
1 similar comment
jonathanpeppers
commented
Aug 18, 2026
@dalexsoto review |
dalexsoto
left a comment
There was a problem hiding this comment.
The JsonReader token consumption, coercion, return values, exceptions, and managed signatures match Android.
Uh oh!
There was an error while loading. Please reload this page.
Refs #247
Sources
android.util.JsonReader(including the existing per-member links for the documented APIs)Validation
[xml].git diff --check.Documented DocIds:
P:Android.Util.JsonReader.HasNext,M:Android.Util.JsonReader.NextBoolean,M:Android.Util.JsonReader.NextDouble,M:Android.Util.JsonReader.NextInt,M:Android.Util.JsonReader.NextLong,M:Android.Util.JsonReader.NextName,M:Android.Util.JsonReader.NextString, andM:Android.Util.JsonReader.Peek.