]> source.dussan.org Git - poi.git/commitdiff
remove some unnecessary casts from copy method
authorPJ Fanning <fanningpj@apache.org>
Sat, 24 Jul 2021 08:10:19 +0000 (08:10 +0000)
committerPJ Fanning <fanningpj@apache.org>
Sat, 24 Jul 2021 08:10:19 +0000 (08:10 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1891763 13f79535-47bb-0310-9956-ffa450edef68

poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFRow.java

index 0484773ecc012536da0a64d6aeefd1a8b38c797e..d0aa2ff86751bdf35f03e5d681323aac4174a42b 100644 (file)
@@ -34,6 +34,7 @@ import org.apache.poi.ss.usermodel.FormulaError;
 import org.apache.poi.ss.usermodel.Row;
 import org.apache.poi.ss.usermodel.helpers.RowShifter;
 import org.apache.poi.ss.util.CellRangeAddress;
+import org.apache.poi.ss.util.CellUtil;
 import org.apache.poi.util.Beta;
 import org.apache.poi.util.Internal;
 import org.apache.poi.xssf.model.StylesTable;
@@ -650,8 +651,7 @@ public class XSSFRow implements Row, Comparable<XSSFRow> {
             // srcRow is blank. Overwrite cells with blank values, blank styles, etc per cell copy policy
             for (Cell destCell : this) {
                 final XSSFCell srcCell = null;
-                // FIXME: remove type casting when copyCellFrom(Cell, CellCopyPolicy) is added to Cell interface
-                ((XSSFCell)destCell).copyCellFrom(srcCell, policy);
+                CellUtil.copyCell(srcCell, destCell, policy);
             }
 
             if (policy.isCopyMergedRegions()) {
@@ -675,10 +675,9 @@ public class XSSFRow implements Row, Comparable<XSSFRow> {
 
         }
         else {
-            for (final Cell c : srcRow){
-                final XSSFCell srcCell = (XSSFCell)c;
-                final XSSFCell destCell = createCell(srcCell.getColumnIndex());
-                destCell.copyCellFrom(srcCell, policy);
+            for (final Cell c : srcRow) {
+                final XSSFCell destCell = createCell(c.getColumnIndex());
+                destCell.copyCellFrom(c, policy);
             }
 
             final int sheetIndex = _sheet.getWorkbook().getSheetIndex(_sheet);