Skip to content

Document Android.Systems StructTimeval conversions - #563

Open
jonathanpeppers wants to merge 2 commits into
mainfrom
jonathanpeppers-document-android-systems-0fa
Open

Document Android.Systems StructTimeval conversions#563
jonathanpeppers wants to merge 2 commits into
mainfrom
jonathanpeppers-document-android-systems-0fa

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Summary

  • Documents StructTimeval.FromMillis, StructTimeval.ToMillis, TvSec, and TvUsec.
  • Uses the official Android reference for the type fields and immutable AOSP source anchors for the conversion behavior missing from that reference.

Sources

Validation

  • XML parsing passed for docs/xml/Android.Systems/StructTimeval.xml.
  • CRLF-only line-ending check passed.
  • git diff --check passed.
  • Selected-member placeholder count is zero for FromMillis, ToMillis, TvSec, and TvUsec; 9 unrelated infrastructure placeholders remain in the file.

Refs #248

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the generated XML documentation for Android.Systems.StructTimeval to describe its millisecond conversion helpers and field semantics, using Android’s reference docs plus immutable AOSP source anchors for behavior details.

Changes:

  • Documented StructTimeval.FromMillis(long) including negative-value conversion behavior and an AOSP source link.
  • Documented StructTimeval.ToMillis() conversion formula and added an AOSP source link.
  • Replaced placeholder <value> docs for TvSec and TvUsec with concrete component descriptions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/xml/Android.Systems/StructTimeval.xml
@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@jonathanpeppers
jonathanpeppers requested review from dalexsoto and removed request for dalexsoto August 15, 2026 15:10
@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

1 similar comment
@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants