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.

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

Fix issue #3728 ParseProblemException #3743

Merged
merged 2 commits into from Nov 1, 2022
Merged

Conversation

jlerbsc
Copy link
Collaborator

@jlerbsc jlerbsc commented Nov 1, 2022

Fixes #3728 . VarType must not be detected on the context of a ClassExpr.
Note : The VarType is detected in the Java10 post processor versus the parser grammar.

@codecov
Copy link

codecov bot commented Nov 1, 2022

Codecov Report

Merging #3743 (7b0123e) into master (99507dc) will increase coverage by 0.008%.
The diff coverage is 90.000%.

Impacted file tree graph

@@               Coverage Diff               @@
##              master     #3743       +/-   ##
===============================================
+ Coverage     57.305%   57.313%   +0.008%     
  Complexity      2671      2671               
===============================================
  Files            635       635               
  Lines          33559     33566        +7     
  Branches        5787      5790        +3     
===============================================
+ Hits           19231     19238        +7     
  Misses         12275     12275               
  Partials        2053      2053               
Flag Coverage Δ
AlsoSlowTests 57.313% <90.000%> (+0.008%) ⬆️
javaparser-core 52.460% <90.000%> (+0.009%) ⬆️
javaparser-symbol-solver 36.730% <70.000%> (+0.007%) ⬆️
jdk-10 57.309% <90.000%> (+0.023%) ⬆️
jdk-11 57.309% <90.000%> (+0.020%) ⬆️
jdk-12 57.306% <90.000%> (+0.020%) ⬆️
jdk-13 57.309% <90.000%> (?)
jdk-14 57.303% <90.000%> (+0.017%) ⬆️
jdk-15 57.306% <90.000%> (+0.005%) ⬆️
jdk-16 57.303% <90.000%> (+0.017%) ⬆️
jdk-8 57.305% <90.000%> (+0.002%) ⬆️
jdk-9 57.303% <90.000%> (+0.002%) ⬆️
macos-latest 57.302% <90.000%> (+0.005%) ⬆️
ubuntu-latest 57.299% <90.000%> (?)
windows-latest 57.293% <90.000%> (+0.005%) ⬆️

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

Impacted Files Coverage Δ
.../validator/postprocessors/Java10PostProcessor.java 94.444% <90.000%> (-5.556%) ⬇️
...ava/com/github/javaparser/ParserConfiguration.java 90.298% <0.000%> (+0.746%) ⬆️

Continue to review full report at Codecov.

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

@jlerbsc jlerbsc merged commit 106f730 into javaparser:master Nov 1, 2022
@jlerbsc jlerbsc added this to the next release milestone Nov 1, 2022
@jlerbsc jlerbsc added the PR: Fixed A PR that offers a fix or correction label Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Fixed A PR that offers a fix or correction
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ParseProblemException
1 participant