Skip to content

Commit 747a9cd

Browse files
committed
Tidy whitespace and format stuff
1 parent 8453573 commit 747a9cd

File tree

7 files changed

+14
-65
lines changed

7 files changed

+14
-65
lines changed

validator-generator/src/main/java/io/avaje/validation/generator/ComponentReader.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,21 +42,17 @@ void read() {
4242
/** Read the existing JsonAdapters from the MetaData annotation of the generated component. */
4343
private void readMetaData(TypeElement moduleType) {
4444
for (final AnnotationMirror annotationMirror : moduleType.getAnnotationMirrors()) {
45-
4645
final MetaDataPrism metaData = MetaDataPrism.getInstance(annotationMirror);
4746
final FactoryPrism metaDataFactory = FactoryPrism.getInstance(annotationMirror);
48-
final AnnotationFactoryPrism metaDataAnnotationFactory =
49-
AnnotationFactoryPrism.getInstance(annotationMirror);
47+
final AnnotationFactoryPrism metaDataAnnotationFactory = AnnotationFactoryPrism.getInstance(annotationMirror);
5048

5149
if (metaData != null) {
52-
5350
metaData.value().stream().map(TypeMirror::toString).forEach(componentMetaData::add);
5451

5552
} else if (metaDataFactory != null) {
56-
5753
metaDataFactory.value().stream().map(TypeMirror::toString).forEach(componentMetaData::add);
58-
} else if (metaDataAnnotationFactory != null) {
5954

55+
} else if (metaDataAnnotationFactory != null) {
6056
metaDataAnnotationFactory.value().stream()
6157
.map(ProcessingContext::asElement)
6258
.forEach(componentMetaData::addAnnotationAdapter);

validator-generator/src/main/java/io/avaje/validation/generator/Constants.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@
22

33
final class Constants {
44

5-
static final String META_INF_COMPONENT =
6-
"META-INF/services/io.avaje.validation.Validator$GeneratedComponent";
5+
static final String META_INF_COMPONENT = "META-INF/services/io.avaje.validation.Validator$GeneratedComponent";
76
public static final String VALID_SPI = "io.avaje.validation.spi.*";
87
public static final String VALIDATOR = "io.avaje.validation.Validator";
98
}

validator-generator/src/main/java/io/avaje/validation/generator/FieldReader.java

Lines changed: 7 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ final class FieldReader {
4040
this.element = element;
4141
if (element instanceof final ExecutableElement executableElement) {
4242
this.rawType = Util.trimAnnotations(executableElement.getReturnType().toString());
43-
4443
} else {
4544
this.rawType = Util.trimAnnotations(element.asType().toString());
4645
}
@@ -152,17 +151,14 @@ private void writeGetValue(Append writer, String suffix) {
152151
}
153152

154153
void writeValidate(Append writer) {
155-
156154
writer.append(" var _$%s = ", fieldName);
157155
writeGetValue(writer, ";");
158156
writer.eol();
159157
if (optionalValidation) {
160-
161158
writer.append(" if(_$%s != null) {", fieldName);
162159
}
163160
writer.append(" %s.validate(_$%s", adapterFieldName, fieldName);
164161
writer.append(", request, \"%s\");", fieldName);
165-
166162
if (optionalValidation) {
167163
writer.append(" }");
168164
}
@@ -179,70 +175,46 @@ public GenericType type() {
179175
}
180176

181177
public void writeConstructor(Append writer) {
182-
183178
writer.append(" this.%s = ", adapterFieldName).eol();
184179

185180
boolean first = true;
186181
for (final var a : annotations.entrySet()) {
187-
188182
if (first) {
189-
writer.append(
190-
" ctx.<%s>adapter(%s.class, %s)",
191-
PrimitiveUtil.wrap(genericType.shortType()), a.getKey().shortName(), a.getValue());
183+
writer.append(" ctx.<%s>adapter(%s.class, %s)", PrimitiveUtil.wrap(genericType.shortType()), a.getKey().shortName(), a.getValue());
192184
first = false;
193185
continue;
194186
}
195-
writer
196-
.eol()
197-
.append(
198-
" .andThen(ctx.adapter(%s.class,%s))",
199-
a.getKey().shortName(), a.getValue());
187+
writer.eol().append(" .andThen(ctx.adapter(%s.class,%s))", a.getKey().shortName(), a.getValue());
200188
}
201189
final var topType = genericType.topType();
202190
if (isBasicType(topType)) {
203-
204191
writer.append(";").eol();
205192
return;
206193
}
207194

208-
if ("java.util.List".equals(genericType.topType())
209-
|| "java.util.Set".equals(genericType.topType())) {
195+
if ("java.util.List".equals(genericType.topType()) || "java.util.Set".equals(genericType.topType())) {
210196
if (isBasicType(genericType.firstParamType())) {
211-
212197
writer.append(";").eol();
213198
return;
214199
}
200+
writer.eol().append(" .list(ctx, %s.class)", Util.shortName(genericType.firstParamType()));
215201

216-
writer
217-
.eol()
218-
.append(" .list(ctx, %s.class)", Util.shortName(genericType.firstParamType()));
219202
} else if ("java.util.Map".equals(genericType.topType())) {
220203
if (isBasicType(genericType.secondParamType())) {
221-
222204
writer.append(";").eol();
223205
return;
224206
}
207+
writer.eol().append(" .map(ctx, %s.class)", Util.shortName(genericType.secondParamType()));
225208

226-
writer
227-
.eol()
228-
.append(" .map(ctx, %s.class)", Util.shortName(genericType.secondParamType()));
229209
} else if (genericType.topType().contains("[]")) {
230210
if (isBasicType(topType)) {
231-
232211
writer.append(";").eol();
233212
return;
234213
}
214+
writer.eol().append(" .array(ctx, %s.class)", Util.shortName(genericType.topType().replace("[]", "")));
235215

236-
writer
237-
.eol()
238-
.append(
239-
" .array(ctx, %s.class)",
240-
Util.shortName(genericType.topType().replace("[]", "")));
241216
} else {
242-
writer
243-
.eol()
244-
.append(
245-
" .andThen(ctx.adapter(%s.class))", Util.shortName(genericType.topType()));
217+
writer.eol().append(" .andThen(ctx.adapter(%s.class))", Util.shortName(genericType.topType()));
246218
}
247219
writer.append(";").eol();
248220
}

validator-generator/src/main/java/io/avaje/validation/generator/GenericTypeParser.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ GenericType parse() {
2222
final char ch = raw.charAt(i);
2323
processChar(ch);
2424
}
25-
2625
return stack.peek();
2726
}
2827

validator-generator/src/main/java/io/avaje/validation/generator/Processor.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,7 @@ private void readModule() {
6565
public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment round) {
6666
readModule();
6767

68-
registerCustomAdapters(
69-
round.getElementsAnnotatedWith(element(AnnotationValidatorPrism.PRISM_TYPE)));
70-
68+
registerCustomAdapters(round.getElementsAnnotatedWith(element(AnnotationValidatorPrism.PRISM_TYPE)));
7169
writeAdapters(round.getElementsAnnotatedWith(element(ValidPojoPrism.PRISM_TYPE)));
7270

7371
Optional.ofNullable(element(ValidPrism.PRISM_TYPE))
@@ -87,11 +85,8 @@ public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment
8785
}
8886

8987
private void registerCustomAdapters(Set<? extends Element> elements) {
90-
9188
for (final var typeElement : ElementFilter.typesIn(elements)) {
92-
9389
AnnotationValidatorPrism.getInstanceOn(typeElement).value();
94-
9590
metaData.addAnnotationAdapter(typeElement);
9691
}
9792
}

validator-generator/src/main/java/io/avaje/validation/generator/SimpleAdapterWriter.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,7 @@ private void writeConstructor() {
6161
writer.append(" /**").eol();
6262
writer.append(" * Construct using Object for generic type parameters.").eol();
6363
writer.append(" */").eol();
64-
writer
65-
.append(" public %sValidationAdapter(ValidationContext ctx) {", adapterShortName)
66-
.eol();
64+
writer.append(" public %sValidationAdapter(ValidationContext ctx) {", adapterShortName).eol();
6765
writer.append(" this(ctx");
6866
for (int i = 0; i < genericParamsCount; i++) {
6967
writer.append(", Object.class");
@@ -83,9 +81,7 @@ private void writeClassEnd() {
8381

8482
private void writeClassStart() {
8583
writer.append("@Generated").eol();
86-
writer.append(
87-
"public final class %sValidationAdapter implements ValidationAdapter<%s> ",
88-
adapterShortName, beanReader.shortName());
84+
writer.append("public final class %sValidationAdapter implements ValidationAdapter<%s> ", adapterShortName, beanReader.shortName());
8985
writer.append("{").eol().eol();
9086
}
9187

validator-generator/src/main/java/io/avaje/validation/generator/TypeReader.java

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -56,14 +56,12 @@ void read(TypeElement type) {
5656
}
5757

5858
for (final FieldReader localField : localFields) {
59-
6059
allFields.add(localField);
6160
allFieldMap.put(localField.fieldName(), localField);
6261
}
6362
}
6463

6564
private void readField(Element element, List<FieldReader> localFields) {
66-
6765
if (includeField(element)) {
6866
localFields.add(new FieldReader(element, genericTypeParams));
6967
}
@@ -86,7 +84,6 @@ private boolean includeField(Element element) {
8684
}
8785

8886
private void readMethod(Element element, TypeElement type, List<FieldReader> localFields) {
89-
9087
final ExecutableElement methodElement = (ExecutableElement) element;
9188
if (methodElement.getModifiers().contains(Modifier.PUBLIC)) {
9289
final List<? extends VariableElement> parameters = methodElement.getParameters();
@@ -119,12 +116,7 @@ private void matchFieldToGetter(FieldReader field) {
119116
&& !field.isPublicField()) {
120117
nonAccessibleField = true;
121118
if (hasJsonAnnotation) {
122-
logError(
123-
"Non accessible field "
124-
+ baseType
125-
+ " "
126-
+ field.fieldName()
127-
+ " with no matching getter?");
119+
logError("Non accessible field " + baseType + " " + field.fieldName() + " with no matching getter?");
128120
} else {
129121
logDebug("Non accessible field " + baseType + " " + field.fieldName());
130122
}

0 commit comments

Comments
 (0)