From: Maxim Valyanskiy Date: Wed, 17 Nov 2010 08:13:40 +0000 (+0000) Subject: dispose xml cursor when iteration over it is done (more) X-Git-Tag: REL_3_8_BETA1~132 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b621f02de0d2d9a30758509d736e8cd201ee98fe;p=poi.git dispose xml cursor when iteration over it is done (more) git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1035947 13f79535-47bb-0310-9956-ffa450edef68 --- diff --git a/src/ooxml/java/org/apache/poi/xslf/usermodel/DrawingParagraph.java b/src/ooxml/java/org/apache/poi/xslf/usermodel/DrawingParagraph.java index e2dcc6cf41..f1e84ce6c1 100644 --- a/src/ooxml/java/org/apache/poi/xslf/usermodel/DrawingParagraph.java +++ b/src/ooxml/java/org/apache/poi/xslf/usermodel/DrawingParagraph.java @@ -44,6 +44,8 @@ public class DrawingParagraph { text.append('\n'); } } + + c.dispose(); return text; } diff --git a/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFCommonSlideData.java b/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFCommonSlideData.java index a926d8b3fc..3716f7e93d 100644 --- a/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFCommonSlideData.java +++ b/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFCommonSlideData.java @@ -69,6 +69,8 @@ public class XSLFCommonSlideData { } } } + + c.dispose(); } return out; diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java index 3d1ee2a58a..4188cbdac1 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java @@ -1098,6 +1098,7 @@ public class XWPFDocument extends POIXMLDocument implements Document, IBody { CTRow row = (CTRow)o; cursor.toParent(); o = cursor.getObject(); + cursor.dispose(); if(! (o instanceof CTTbl)){ return null; } diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFFooter.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFFooter.java index 73834d7672..feaf9f7dd0 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFFooter.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFFooter.java @@ -66,6 +66,7 @@ public class XWPFFooter extends XWPFHeaderFooter { bodyElements.add(t); } } + cursor.dispose(); getAllPictures(); } @@ -125,6 +126,7 @@ public class XWPFFooter extends XWPFHeaderFooter { bodyElements.add(t); } } + cursor.dispose(); getAllPictures(); } catch (IOException e) { // TODO Auto-generated catch block diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeader.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeader.java index d49aa5d7ec..07f59afffe 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeader.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeader.java @@ -67,6 +67,7 @@ public class XWPFHeader extends XWPFHeaderFooter { tables.add(t); } } + cursor.dispose(); getAllPictures(); } @@ -131,6 +132,7 @@ public class XWPFHeader extends XWPFHeaderFooter { bodyElements.add(t); } } + cursor.dispose(); getAllPictures(); } catch (IOException e) { // TODO Auto-generated catch block diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeaderFooter.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeaderFooter.java index 5aedc15f9b..96274be21a 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeaderFooter.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFHeaderFooter.java @@ -504,6 +504,7 @@ public abstract class XWPFHeaderFooter extends POIXMLDocumentPart implements IBo bodyElements.add(t); } } + cursor.dispose(); getAllPictures(); } @@ -521,6 +522,7 @@ public abstract class XWPFHeaderFooter extends POIXMLDocumentPart implements IBo CTRow row = (CTRow)o; cursor.toParent(); o = cursor.getObject(); + cursor.dispose(); if(! (o instanceof CTTbl)){ return null; } diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java index 2e7e16f3ed..2833695e92 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFParagraph.java @@ -107,6 +107,8 @@ public class XWPFParagraph implements IBodyElement{ } } } + + c.dispose(); // Look for bits associated with the runs for(XWPFRun run : runs) { @@ -1159,6 +1161,8 @@ public class XWPFParagraph implements IBodyElement{ else candCharPos=0; } + + c.dispose(); } return null; } diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java index 067f2c0882..0e8bbb20ad 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java @@ -628,6 +628,8 @@ public class XWPFRun { } } } + + c.dispose(); // Any picture text? if(pictureText != null && pictureText.length() > 0) { diff --git a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java index 656bf307a8..9ffb65a340 100644 --- a/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java +++ b/src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java @@ -65,6 +65,7 @@ public class XWPFTableCell implements IBody { bodyElements.add(t); } } + cursor.dispose(); } @@ -336,6 +337,7 @@ public class XWPFTableCell implements IBody { CTRow row = (CTRow)o; cursor.toParent(); o = cursor.getObject(); + cursor.dispose(); if(! (o instanceof CTTbl)){ return null; }