You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by on...@apache.org on 2017/11/12 01:32:43 UTC

svn commit: r1814998 - /poi/trunk/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java

Author: onealj
Date: Sun Nov 12 01:32:43 2017
New Revision: 1814998

URL: http://svn.apache.org/viewvc?rev=1814998&view=rev
Log:
make tableRows final; initialize where defined rather than in constructor.

Modified:
    poi/trunk/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java

Modified: poi/trunk/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java?rev=1814998&r1=1814997&r2=1814998&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java Sun Nov 12 01:32:43 2017
@@ -74,7 +74,7 @@ public class XWPFTable implements IBodyE
     }
 
     protected StringBuilder text = new StringBuilder(64);
-    protected List<XWPFTableRow> tableRows;
+    protected final List<XWPFTableRow> tableRows = new ArrayList<>();
 
     // Unused: UUF_UNUSED_PUBLIC_OR_PROTECTED_FIELD
     //protected List<String> styleIDs;
@@ -98,8 +98,6 @@ public class XWPFTable implements IBodyE
         this.part = part;
         this.ctTbl = table;
 
-        tableRows = new ArrayList<>();
-
         // is an empty table: I add one row and one column as default
         if (table.sizeOfTrArray() == 0)
             createEmptyTable(table);



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org