Skip to content

Test relocation of nested class signatures#2108

Merged
Goooler merged 1 commit into
mainfrom
g/20260715/test-nested-for-asm
Jul 15, 2026
Merged

Test relocation of nested class signatures#2108
Goooler merged 1 commit into
mainfrom
g/20260715/test-nested-for-asm

Conversation

@Goooler

@Goooler Goooler commented Jul 15, 2026

Copy link
Copy Markdown
Member

Refs #1902.

@Goooler Goooler marked this pull request as ready for review July 15, 2026 08:23
@Goooler Goooler requested a review from Copilot July 15, 2026 08:23

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.

Pull request overview

Adds a regression test to ensure bytecode signature relocation correctly handles nested/inner class generic signatures (related to #1902’s Class-File API work and signature handling).

Changes:

  • Add a new test (nestedClassSignatureIsRelocated) covering relocation of a nested/inner generic signature.
  • Introduce FixtureGenericOuter/FixtureInner and a fixture method (methodWithNestedGeneric) to generate the target signature in compiled bytecode.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Goooler Goooler merged commit e37770c into main Jul 15, 2026
5 checks passed
@Goooler Goooler deleted the g/20260715/test-nested-for-asm branch July 15, 2026 08:28
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