Skip to content

[DeadCode] Retarget RemoveParentDelegatingConstructorRector to Class_ node - #8751

Merged
TomasVotruba merged 3 commits into
mainfrom
retarget-remove-parent-delegating-constructor-to-class
Oct 21, 2026
Merged

TomasVotruba merged 3 commits into
mainfrom
retarget-remove-parent-delegating-constructor-to-class

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Subscribe to Class_ and pull the __construct method once via getMethod(), instead of subscribing to every ClassMethod and filtering by name on each visit.

Behavior is unchanged - the constructor is removed from the class stmts when it only delegates to the parent with identical args. All 15 fixture tests stay green.

@TomasVotruba
TomasVotruba enabled auto-merge (squash) October 21, 2026 20:38
@TomasVotruba
TomasVotruba merged commit 2fff0bd into main Oct 21, 2026
49 checks passed
@TomasVotruba
TomasVotruba deleted the retarget-remove-parent-delegating-constructor-to-class branch October 21, 2026 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant