Skip to content

Document MethodHandle reference kinds - #441

Open
jonathanpeppers wants to merge 1 commit into
mainfrom
jonathanpeppers-document-java-invoke-apis-9c9
Open

Document MethodHandle reference kinds#441
jonathanpeppers wants to merge 1 commit into
mainfrom
jonathanpeppers-document-java-invoke-apis-9c9

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Summary

  • Documented the Java.Lang.Invoke.ReferenceKind enum and all nine reference-kind values.
  • Kept the change limited to source-backed method-handle field, method, and constructor access behavior.

Refs #201

Sources

Validation

  • XML parse of docs/xml/Java.Lang.Invoke/ReferenceKind.xml
  • Confirmed CRLF line endings and no bare LF characters
  • git diff --check
  • Confirmed all 10 placeholders in ReferenceKind.xml were replaced

Refs #201

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 replaces placeholder XML documentation for Java.Lang.Invoke.ReferenceKind with concrete summaries describing the MethodHandleInfo reference kinds used for field, method, and constructor access.

Changes:

  • Documented the Java.Lang.Invoke.ReferenceKind enum summary.
  • Replaced all nine enum value <summary> placeholders with descriptions matching their reference-kind behavior.

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

@jonathanpeppers

Copy link
Copy Markdown
Member Author

@dalexsoto review

@dalexsoto dalexsoto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The documented API behavior matches the current contracts; no blocking issues found.

auto-merge was automatically disabled August 15, 2026 02:21

Pull Request is not mergeable

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.

3 participants