Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[java] Use type inference to resolve complicated cases of CollectionTypeMismatch #6037

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
Draft
oowekyala wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

[java] Use type inference to resolve complicated cases of CollectionTypeMismatch #6037

oowekyala wants to merge 13 commits into from

Conversation

@oowekyala
Copy link
Member

@oowekyala oowekyala commented Sep 6, 2025 •
edited
Loading

See #6006 (comment)

This uses type inference to resolve the more complicated cases around generics.

Describe the PR

Related issues

Ready?

  • Missing unit tests for InferenceCtx around ivars with null tvar
  • Missing unit tests for CollectionTypeMismatch for new cases that we can check
  • Disable checkstyle MissingBreakInSwitch as our PMD rule NonExhaustiveSwitch is better, it checks for exhaustivity. Disable checkstyle MissingSwitchDefault rule build-tools#98
  • Passing all unit tests
  • Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • Added (in-code) documentation (if needed)

@oowekyala oowekyala mentioned this pull request Sep 6, 2025
Merged
@pmd-actions-helper
Copy link
Contributor

pmd-actions-helper bot commented Sep 6, 2025 •
edited
Loading

Documentation Preview

Compared to main:
This changeset changes 0 violations,
introduces 7 new violations, 0 new errors and 0 new configuration errors,
removes 1 violations, 0 errors and 0 configuration errors.

Regression Tester Report

(comment created at 2025-10-21 09:43:03+00:00 for decf795)

adangel added a commit to pmd/build-tools that referenced this pull request Oct 16, 2025
@adangel
Disable checkstyle MissingSwitchDefault rule (#98)
385d88e 
This check caused a checkstyle failure on
pmd/pmd#6037 even though the switch is
exhaustive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adangel adangel adangel left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@oowekyala @adangel