Skip to content

Commit f662334

Browse files
committed
Format only
1 parent f31ae57 commit f662334

File tree

2 files changed

+28
-38
lines changed

2 files changed

+28
-38
lines changed

jsonb-generator/src/main/java/io/avaje/jsonb/generator/FieldReader.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,11 @@ final class FieldReader {
2525
private boolean isCreatorParam;
2626

2727
FieldReader(
28-
Element element,
29-
NamingConvention namingConvention,
30-
TypeSubTypeMeta subType,
31-
List<String> genericTypeParams,
32-
Integer frequency) {
28+
Element element,
29+
NamingConvention namingConvention,
30+
TypeSubTypeMeta subType,
31+
List<String> genericTypeParams,
32+
Integer frequency) {
3333

3434
this(element, namingConvention, subType, genericTypeParams, frequency, false);
3535
}
@@ -41,6 +41,7 @@ final class FieldReader {
4141
List<String> genericTypeParams,
4242
Integer frequency,
4343
boolean jsonCreatorPresent) {
44+
4445
this.element = element;
4546
num = frequency == 0 ? "" : frequency.toString();
4647
addSubType(subType);
@@ -297,7 +298,7 @@ List<String> aliases() {
297298
return aliases;
298299
}
299300

300-
public Element element() {
301+
Element element() {
301302
return element;
302303
}
303304
}

jsonb-generator/src/main/java/io/avaje/jsonb/generator/TypeReader.java

Lines changed: 21 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,8 @@ final class TypeReader {
4343
private boolean nonAccessibleField;
4444

4545
private final Map<String, Element> mixInFields;
46-
4746
private final String typePropertyKey;
48-
4947
private final Map<String, Integer> frequencyMap = new HashMap<>();
50-
5148
private final List<MethodProperty> methodProperties = new ArrayList<>();
5249

5350
private boolean optional;
@@ -141,10 +138,9 @@ void read(TypeElement type) {
141138
for (var param : constructor.getParams()) {
142139
var name = param.name();
143140
var element = param.element();
144-
var matchingField =
145-
localFields.stream()
146-
.filter(f -> f.propertyName().equals(name) || f.fieldName().equals(name))
147-
.findFirst();
141+
var matchingField = localFields.stream()
142+
.filter(f -> f.propertyName().equals(name) || f.fieldName().equals(name))
143+
.findFirst();
148144
matchingField.ifPresentOrElse(f -> f.readParam(element), () -> readField(element, localFields));
149145
}
150146
}
@@ -163,7 +159,6 @@ void read(TypeElement type) {
163159
} else {
164160
commonField.addSubType(currentSubType);
165161
}
166-
167162
if (commonField == null && currentSubType != null) {
168163
localField.setSubTypeField();
169164
}
@@ -174,22 +169,19 @@ void read(TypeElement type) {
174169
private void readField(Element element, List<FieldReader> localFields) {
175170
final Element mixInField = mixInFields.get(element.getSimpleName().toString());
176171
if (mixInField != null && APContext.types().isSameType(mixInField.asType(), element.asType())) {
177-
178172
var mixinModifiers = new HashSet<>(mixInField.getModifiers());
179173
var modifiers = new HashSet<>(mixInField.getModifiers());
180174

181175
Arrays.stream(Modifier.values())
182-
.filter(m -> m != Modifier.PRIVATE || m != Modifier.PROTECTED || m != Modifier.PUBLIC)
183-
.forEach(
184-
m -> {
185-
modifiers.remove(m);
186-
mixinModifiers.remove(m);
187-
});
176+
.filter(m -> m != Modifier.PRIVATE || m != Modifier.PROTECTED || m != Modifier.PUBLIC)
177+
.forEach(m -> {
178+
modifiers.remove(m);
179+
mixinModifiers.remove(m);
180+
});
188181

189182
if (!modifiers.equals(mixinModifiers)) {
190183
APContext.logError(mixInField, "mixIn fields must have the same modifiers as the target class");
191184
}
192-
193185
element = mixInField;
194186
}
195187
if (element.asType().toString().contains("java.util.Optional")) {
@@ -466,24 +458,21 @@ private MethodReader determineConstructor() {
466458
}
467459

468460
// find the right constructor
469-
var contructorFields =
470-
allFields.stream()
471-
.filter(FieldReader::includeFromJson)
472-
.filter(this::hasNoSetter)
473-
.map(f -> f.element().asType().toString())
474-
.map(Util::trimAnnotations)
475-
.collect(toSet());
461+
var constructorFields = allFields.stream()
462+
.filter(FieldReader::includeFromJson)
463+
.filter(this::hasNoSetter)
464+
.map(f -> f.element().asType().toString())
465+
.map(Util::trimAnnotations)
466+
.collect(toSet());
476467

477468
return allPublic.stream()
478-
.filter(c -> c.getParams().size() == contructorFields.size())
479-
.filter(
480-
c ->
481-
c.getParams().stream()
482-
.map(p -> p.element().asType().toString())
483-
.map(Util::trimAnnotations)
484-
.allMatch(contructorFields::contains))
485-
.findFirst()
486-
.orElseGet(this::largest);
469+
.filter(c -> c.getParams().size() == constructorFields.size())
470+
.filter(c -> c.getParams().stream()
471+
.map(p -> p.element().asType().toString())
472+
.map(Util::trimAnnotations)
473+
.allMatch(constructorFields::contains))
474+
.findFirst()
475+
.orElseGet(this::largest);
487476
}
488477

489478
private MethodReader largest() {

0 commit comments

Comments
 (0)