Skip to content

Unified: Implement ClassLikeDeclaration.toString - #22698

Merged
hvitved merged 1 commit into
github:mainfrom
hvitved:unified/class-like-decl-to-string
Sep 29, 2026
Merged

hvitved merged 1 commit into
github:mainfrom
hvitved:unified/class-like-decl-to-string

Conversation

@hvitved

@hvitved hvitved commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@hvitved
hvitved marked this pull request as ready for review September 29, 2026 14:25
@hvitved
hvitved requested a review from a team as a code owner September 29, 2026 14:25
Copilot AI balanced review requested due to automatic review settings September 29, 2026 14:25
@hvitved hvitved added the no-change-note-required This PR does not need a change note label Sep 29, 2026
@hvitved
hvitved requested a review from asgerf September 29, 2026 14:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Extensions stringify without their target type, making distinct extensions indistinguishable.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Implements readable string representations for unified class-like declarations.

Changes:

  • Derives declaration kind and name in ClassLikeDeclaration.toString.
  • Updates generated control-flow expectations.
File Description
FacadeAst.qll Adds class-like declaration formatting.
basicblock-slices.expected Updates generated labels.

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

result in ["class", "struct", "interface", "enum", "actor", "extension", "protocol"]
}

override string toString() { result = concat(this.getKind() + " ") + concat(this.getName()) }
@hvitved
hvitved merged commit 22e050f into github:main Sep 29, 2026
15 of 16 checks passed
@hvitved
hvitved deleted the unified/class-like-decl-to-string branch September 29, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-change-note-required This PR does not need a change note Unified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants