Skip to content

Commit dda048b

Browse files
committed
Use tryCast()
1 parent 1df44df commit dda048b

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

src/main/java/org/zwobble/mammoth/internal/docx/StatefulBodyXmlReader.java

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
import static org.zwobble.mammoth.internal.docx.ReadResult.*;
1616
import static org.zwobble.mammoth.internal.docx.Uris.uriToZipEntryName;
17+
import static org.zwobble.mammoth.internal.util.Casts.tryCast;
1718
import static org.zwobble.mammoth.internal.util.Iterables.lazyFilter;
1819
import static org.zwobble.mammoth.internal.util.Iterables.tryGetLast;
1920
import static org.zwobble.mammoth.internal.util.Lists.eagerConcat;
@@ -546,7 +547,7 @@ private ReadResult calculateRowspans(List<DocumentElement> rows) {
546547

547548
private Optional<String> checkTableRows(List<DocumentElement> rows) {
548549
for (DocumentElement rowElement : rows) {
549-
Optional<TableRow> row = Casts.tryCast(TableRow.class, rowElement);
550+
Optional<TableRow> row = tryCast(TableRow.class, rowElement);
550551
if (!row.isPresent()) {
551552
return Optional.of("unexpected non-row element in table, cell merging may be incorrect");
552553
} else {
@@ -775,11 +776,9 @@ private <T extends DocumentElement> Function<DocumentElement, DocumentElement> t
775776
);
776777
}
777778

778-
if (elementClass.isInstance(element)) {
779-
return transform.apply((T) element);
780-
} else {
781-
return element;
782-
}
779+
return tryCast(elementClass, element)
780+
.map(transform)
781+
.orElse(element);
783782
};
784783
}
785784

0 commit comments

Comments
 (0)