Skip to content

Conversation

@zosrothko
Copy link
Member

Hi John

This PR provides the support of the generation of JavaCC tool for the C++ target. There is a new script variable named 'language' defaulted to Language.Java. Thus all code & tests for the Java generation are identical to your latest master.

Signed-off-by: FrancisANDRE <zosrothko@orange.fr>
…uage.class)

Signed-off-by: FrancisANDRE <zosrothko@orange.fr>
Signed-off-by: FrancisANDRE <zosrothko@orange.fr>
@codecov-io
Copy link

codecov-io commented Feb 7, 2017

Codecov Report

Merging #40 into master will increase coverage by -8.23%.

@@             Coverage Diff             @@
##             master     #40      +/-   ##
===========================================
- Coverage     87.73%   79.5%   -8.23%     
- Complexity        0     152     +152     
===========================================
  Files            23      24       +1     
  Lines           432     488      +56     
  Branches         39      44       +5     
===========================================
+ Hits            379     388       +9     
- Misses           49      86      +37     
- Partials          4      14      +10
Impacted Files Coverage Δ Complexity Δ
...glinc/gradle/plugins/javacc/CompileJjtreeTask.java 23.52% <ø> (ø) 2 <ø> (+2)
...glinc/gradle/plugins/javacc/CompileJavaccTask.java 23.52% <ø> (ø) 2 <ø> (+2)
...oglinc/gradle/plugins/javacc/CompileJjdocTask.java 25% <ø> (ø) 2 <ø> (+2)
...ns/javacc/compiler/NonJavaccSourceFileVisitor.java 78.94% <ø> (-21.06%) 5 <2> (+5)
...mpilationresults/CompiledJavaccFilesDirectory.java 100% <100%> (ø) 4 <1> (+4)
...onresults/CompiledJavaccFilesDirectoryFactory.java 100% <100%> (ø) 9 <ø> (+9)
...ava/ca/coglinc/gradle/plugins/javacc/Language.java 100% <100%> (ø) 1 <1> (?)
...linc/gradle/plugins/javacc/AbstractJavaccTask.java 61.53% <25%> (-6.65%) 6 <1> (+6)
...e/plugins/javacc/JavaToJavaccDependencyAction.java 57.14% <56.25%> (-42.86%) 13 <12> (+13)
.../javacc/compilationresults/CompiledJavaccFile.java 73.86% <73.86%> (-18.2%) 19 <19> (+19)
... and 2 more

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 f001e5f...1d30c2e. Read the comment docs.

@zosrothko
Copy link
Member Author

Hi John..; Would you mind to have a look at this PR.. I need it published for adding tests on JavaCC c++ generation. Thank in advance.

Conflicts:
	subprojects/plugin/src/test/java/ca/coglinc/gradle/plugins/javacc/programexecution/JjdocProgramInvokerTest.java
Conflicts:
	subprojects/plugin/src/test/java/ca/coglinc/gradle/plugins/javacc/programexecution/JjdocProgramInvokerTest.java
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