[java] Fix parser error (issue 1530) #110

Merged
merged 2 commits into from Oct 16, 2016

Projects

None yet

2 participants

@jsotuyod
Member
  • Added a test case to show the regression
  • Changed the grammar to handle it properly

I don't fully understand why JavaCC fails to properly use the syntactic lookahead, but it's fully working. All tests green.

jsotuyod added some commits Oct 12, 2016
@jsotuyod jsotuyod Add failing test scenario a50c781
@jsotuyod jsotuyod Replace syntactic lookahead with semantic one
 - They are the same, but JavaCC seems to handle this one better.
a10f98e
@adangel adangel changed the title from Fix issue 1530 to [java] Fix parser error (issue 1530) Oct 16, 2016
@adangel
Member
adangel commented Oct 16, 2016

Thanks!

@adangel adangel merged commit a10f98e into pmd:master Oct 16, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jsotuyod jsotuyod deleted the Monits:issue-1530 branch Oct 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment