Skip to content

Commit 601cdde

Browse files
committed
update processor
1 parent 9f87261 commit 601cdde

4 files changed

Lines changed: 10 additions & 22 deletions

File tree

compiler/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ shadowJar {
5252

5353
dependencies {
5454
def jbock = project(':jbock')
55-
def simple_component = 'io.github.jbock-java:simple-component:1.014'
55+
def simple_component = 'io.github.jbock-java:simple-component:1.015'
5656
def javapoet = 'io.github.jbock-java:javapoet:1.15'
5757
implementation(javapoet)
5858
implementation('io.github.jbock-java:auto-common:1.2.3')
5959
shadow(jbock)
6060
compileOnly(simple_component)
61-
annotationProcessor('io.github.jbock-java:simple-component-compiler:1.014')
61+
annotationProcessor('io.github.jbock-java:simple-component-compiler:1.015')
6262
testImplementation('io.github.jbock-java:compile-testing:0.19.12')
6363
testImplementation('org.junit.jupiter:junit-jupiter:5.9.1')
6464
testImplementation('org.mockito:mockito-core:4.8.0')

compiler/src/main/java/net/jbock/processor/CommandStep.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
import net.jbock.common.TypeTool;
1010
import net.jbock.common.Util;
1111
import net.jbock.common.ValidationFailure;
12-
import net.jbock.validate.ValidateComponent;
12+
import net.jbock.validate.ValidateComponent_Impl;
1313
import net.jbock.writing.ContextComponent;
1414

1515
import javax.annotation.processing.Messager;
@@ -72,7 +72,11 @@ public Set<? extends Element> process(Map<String, Set<Element>> elementsByAnnota
7272
}
7373

7474
private void processSourceElement(SourceElement sourceElement) {
75-
ValidateComponent.generate(util, tool, sourceElement)
75+
ValidateComponent_Impl.builder()
76+
.util(util)
77+
.tool(tool)
78+
.sourceElement(sourceElement)
79+
.build().commandProcessor().generate()
7680
.map(ContextComponent::parserClass)
7781
.ifLeftOrElse(
7882
this::printFailures,

compiler/src/main/java/net/jbock/validate/CommandProcessor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
* This class is responsible for item validation.
2626
* If validation succeeds, a {@link CommandRepresentation} is created.
2727
*/
28-
final class CommandProcessor {
28+
public final class CommandProcessor {
2929

3030
private final ItemsFactory itemsFactory;
3131
private final SourceElement sourceElement;
@@ -47,7 +47,7 @@ final class CommandProcessor {
4747
this.varargsParameterValidator = varargsParameterValidator;
4848
}
4949

50-
Either<List<ValidationFailure>, CommandRepresentation> generate() {
50+
public Either<List<ValidationFailure>, CommandRepresentation> generate() {
5151
return itemsFactory.createItems()
5252
.filter(this::checkDuplicateDescriptionKeys)
5353
.flatMap(items -> {

compiler/src/main/java/net/jbock/validate/ValidateComponent.java

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,11 @@
33
import io.jbock.simple.Component;
44
import io.jbock.simple.Named;
55
import io.jbock.simple.Provides;
6-
import io.jbock.util.Either;
76
import net.jbock.common.TypeTool;
87
import net.jbock.common.Util;
9-
import net.jbock.common.ValidationFailure;
108
import net.jbock.convert.match.Matcher;
119
import net.jbock.processor.SourceElement;
12-
import net.jbock.writing.CommandRepresentation;
1310

14-
import java.util.List;
1511
import java.util.Set;
1612

1713
@Component
@@ -30,18 +26,6 @@ interface Builder {
3026
ValidateComponent build();
3127
}
3228

33-
static Either<List<ValidationFailure>, CommandRepresentation> generate(
34-
Util util,
35-
TypeTool tool,
36-
SourceElement sourceElement) {
37-
ValidateComponent component = ValidateComponent_Impl.builder()
38-
.util(util)
39-
.tool(tool)
40-
.sourceElement(sourceElement)
41-
.build();
42-
return component.commandProcessor().generate();
43-
}
44-
4529
@Provides
4630
static Set<Matcher> allMatchers(
4731
@Named("optional") Matcher optionalMatcher,

0 commit comments

Comments
 (0)