Skip to content

Fix resolution for method refs used as varargs#4759

Merged
jlerbsc merged 1 commit intojavaparser:masterfrom
johannescoetzee:johannes/fix-method-ref-varargs
May 19, 2025
Merged

Fix resolution for method refs used as varargs#4759
jlerbsc merged 1 commit intojavaparser:masterfrom
johannescoetzee:johannes/fix-method-ref-varargs

Conversation

@johannescoetzee
Copy link
Copy Markdown
Collaborator

This PR is effectively the same thing as #4752, but for fixing a crash where a vararg MethodReferenceExpr is resolved as a method usage. The first change fixes an IndexOutOfBounds crash, while the second fixes the same issue as in the linked PR where the resulting type is an ArrayType(...) which breaks the rest of the resolution logic.

@codecov
Copy link
Copy Markdown

codecov bot commented May 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.326%. Comparing base (ab4e404) to head (14479e3).
Report is 3 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##              master     #4759       +/-   ##
===============================================
+ Coverage     58.321%   58.326%   +0.004%     
- Complexity      2512      2514        +2     
===============================================
  Files            671       671               
  Lines          38780     38782        +2     
  Branches        7040      7041        +1     
===============================================
+ Hits           22617     22620        +3     
  Misses         13271     13271               
+ Partials        2892      2891        -1     
Flag Coverage Δ
AlsoSlowTests 58.326% <100.000%> (+0.004%) ⬆️
javaparser-core 58.326% <100.000%> (+0.004%) ⬆️
javaparser-symbol-solver 58.326% <100.000%> (+0.004%) ⬆️
jdk-10 57.867% <100.000%> (+0.004%) ⬆️
jdk-11 57.876% <100.000%> (+0.004%) ⬆️
jdk-12 57.865% <100.000%> (+0.004%) ⬆️
jdk-13 57.876% <100.000%> (+0.004%) ⬆️
jdk-14 58.113% <100.000%> (+0.004%) ⬆️
jdk-15 58.123% <100.000%> (+0.004%) ⬆️
jdk-16 58.087% <100.000%> (-0.006%) ⬇️
jdk-17 58.252% <100.000%> (+0.004%) ⬆️
jdk-18 58.252% <100.000%> (+0.004%) ⬆️
jdk-8 57.878% <100.000%> (+0.004%) ⬆️
jdk-9 57.867% <100.000%> (+0.004%) ⬆️
macos-latest 58.318% <100.000%> (+0.004%) ⬆️
ubuntu-latest 58.302% <100.000%> (+0.004%) ⬆️
windows-latest 58.307% <100.000%> (+0.004%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...er/symbolsolver/javaparsermodel/TypeExtractor.java 72.000% <100.000%> (+0.523%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bfa3340...14479e3. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jlerbsc jlerbsc merged commit 1da32e8 into javaparser:master May 19, 2025
35 checks passed
@jlerbsc jlerbsc added this to the next release milestone May 19, 2025
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