Skip to content

Commit 6c76839

Browse files
authored
Merge pull request #147 from avaje/feature/module-info-generator
Add a try catch in generator to improve the error reporting
2 parents 59631a5 + 1b1e49c commit 6c76839

File tree

2 files changed

+16
-12
lines changed

2 files changed

+16
-12
lines changed

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

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -344,18 +344,22 @@ private void writeViewBuild(Append writer) {
344344

345345
@Override
346346
public void writeToJson(Append writer) {
347-
final String varName = Util.initLower(shortName);
348-
writer.eol();
349-
writer.append(" @Override").eol();
350-
writer.append(" public void toJson(JsonWriter writer, %s %s) {", shortName, varName).eol();
351-
writer.append(" writer.beginObject(names);").eol();
352-
if (hasSubTypes) {
353-
writeToJsonForSubtypes(writer, varName);
354-
} else {
355-
writeToJsonForType(writer, varName, " ", null);
347+
try {
348+
final String varName = Util.initLower(shortName);
349+
writer.eol();
350+
writer.append(" @Override").eol();
351+
writer.append(" public void toJson(JsonWriter writer, %s %s) {", shortName, varName).eol();
352+
writer.append(" writer.beginObject(names);").eol();
353+
if (hasSubTypes) {
354+
writeToJsonForSubtypes(writer, varName);
355+
} else {
356+
writeToJsonForType(writer, varName, " ", null);
357+
}
358+
writer.append(" writer.endObject();").eol();
359+
writer.append(" }").eol();
360+
} catch (RuntimeException e) {
361+
throw new IllegalStateException("Error writing toJson() on " + type, e);
356362
}
357-
writer.append(" writer.endObject();").eol();
358-
writer.append(" }").eol();
359363
}
360364

361365
private void writeToJsonForSubtypes(Append writer, String varName) {

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ private void writeGetValue(Append writer, String varName, String suffix) {
232232
} else if (publicField) {
233233
writer.append("%s.%s%s", varName, fieldName, suffix);
234234
} else {
235-
throw new IllegalStateException("Field" + fieldName + " is inaccessible. Add a getter or make the field public.");
235+
throw new IllegalStateException("Field " + fieldName + " is inaccessible. Add a getter or make the field public.");
236236
}
237237
}
238238

0 commit comments

Comments
 (0)