Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Backport "Implement -Xlint:private-shadow, type-parameter-shadow" to LTS #20685

Merged
merged 11 commits into from
Jun 20, 2024
Merged

Backport "Implement -Xlint:private-shadow, type-parameter-shadow" to LTS #20685

merged 11 commits into from
Jun 20, 2024

Conversation

WojciechMazur
Copy link
Contributor

Backports #17622 to the LTS branch.

PR submitted by the release tooling.
[skip ci]

schuetzcarl and others added 11 commits June 20, 2024 15:10
@schuetzcarl @WojciechMazur
Implement -Xlint:private-shadow, type-parameter-shadow
057d8f3 
Respectively warn about :
- a private field or a class parameter that shadows a superclass field
- a local type parameter that shadows a type already in the scope

Fixes : #17612 and #17613

[Cherry-picked e5fd477]
@schuetzcarl @WojciechMazur
Move out miniphase
f782a66 
[Cherry-picked 1f15b29]
@WojciechMazur
Fix Exception in CheckUnused isOverriden() helper
76b797f 
[Cherry-picked 175d4f3][modified]
@schuetzcarl @WojciechMazur
Warn instead of fail for invalid -Xlint args
d09614a 
[Cherry-picked 07f6581]
@schuetzcarl @WojciechMazur
Lint MegaPhase after ExtractSemanticDB
ce9003d 
[Cherry-picked 297183c]
@schuetzcarl @WojciechMazur
Adjust export and synth. case class constructor false-positive cases …
a6e3d7e 
…& Add Tests

[Cherry-picked 46f2db6]
@schuetzcarl @WojciechMazur
Move tests from neg-custom-args/fatal-warnings to neg
c431d61 
[Cherry-picked fb8426a]
@schuetzcarl @WojciechMazur
Delete tests in old folder
830ed0f 
[Cherry-picked 2edd869]
@schuetzcarl @WojciechMazur
Change options line for tests/neg folder
2f81430 
[Cherry-picked 1db2afb]
@schuetzcarl @WojciechMazur
Apply review changes
7b1b898 
[Cherry-picked e3c984e]
@schuetzcarl @WojciechMazur
Separate CheckUnused and CheckShadowing into two MegaPhases
ddc505a 
[Cherry-picked 0f4f7bf]
Base automatically changed from lts-18560 to lts-3.3 June 20, 2024 14:55
@WojciechMazur
Copy link
Contributor Author

No regressions detected in the community build up to lts-18731.

Reference

@WojciechMazur WojciechMazur merged commit b22e8e5 into lts-3.3 Jun 20, 2024
19 checks passed
@WojciechMazur WojciechMazur deleted the lts-17622 branch June 20, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@WojciechMazur @schuetzcarl