You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by fa...@apache.org on 2021/12/28 10:51:24 UTC
svn commit: r1896472 - in /poi/trunk/poi-examples/src/main/java/org/apache/poi/examples: ss/LoadEmbedded.java xslf/AddVideoToPptx.java.txt xslf/LinkVideoToPptx.java
Author: fanningpj
Date: Tue Dec 28 10:51:24 2021
New Revision: 1896472
URL: http://svn.apache.org/viewvc?rev=1896472&view=rev
Log:
close cursors in finally blocks
Modified:
poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/ss/LoadEmbedded.java
poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/AddVideoToPptx.java.txt
poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/LinkVideoToPptx.java
Modified: poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/ss/LoadEmbedded.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/ss/LoadEmbedded.java?rev=1896472&r1=1896471&r2=1896472&view=diff
==============================================================================
--- poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/ss/LoadEmbedded.java (original)
+++ poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/ss/LoadEmbedded.java Tue Dec 28 10:51:24 2021
@@ -101,33 +101,46 @@ public final class LoadEmbedded {
String contentType = pPart.getContentType();
if (contentType.equals("application/vnd.ms-excel")) {
// Excel Workbook - either binary or OpenXML
- HSSFWorkbook embeddedWorkbook = new HSSFWorkbook(pPart.getInputStream());
- embeddedWorkbook.close();
+ try (InputStream stream = pPart.getInputStream()) {
+ HSSFWorkbook embeddedWorkbook = new HSSFWorkbook(stream);
+ embeddedWorkbook.close();
+ }
} else if (contentType.equals("application/vnd.openxmlformats-officedocument.spreadsheetml.sheet")) {
// Excel Workbook - OpenXML file format
- XSSFWorkbook embeddedWorkbook = new XSSFWorkbook(pPart.getInputStream());
- embeddedWorkbook.close();
+ try (InputStream stream = pPart.getInputStream()) {
+ XSSFWorkbook embeddedWorkbook = new XSSFWorkbook(stream);
+ embeddedWorkbook.close();
+ }
} else if (contentType.equals("application/msword")) {
// Word Document - binary (OLE2CDF) file format
- HWPFDocument document = new HWPFDocument(pPart.getInputStream());
- document.close();
+ try (InputStream stream = pPart.getInputStream()) {
+ HWPFDocument document = new HWPFDocument(stream);
+ document.close();
+ }
} else if (contentType.equals("application/vnd.openxmlformats-officedocument.wordprocessingml.document")) {
// Word Document - OpenXML file format
- XWPFDocument document = new XWPFDocument(pPart.getInputStream());
- document.close();
+ try (InputStream stream = pPart.getInputStream()) {
+ XWPFDocument document = new XWPFDocument(stream);
+ document.close();
+ }
} else if (contentType.equals("application/vnd.ms-powerpoint")) {
// PowerPoint Document - binary file format
- HSLFSlideShow slideShow = new HSLFSlideShow(pPart.getInputStream());
- slideShow.close();
+ try (InputStream stream = pPart.getInputStream()) {
+ HSLFSlideShow slideShow = new HSLFSlideShow(stream);
+ slideShow.close();
+ }
} else if (contentType.equals("application/vnd.openxmlformats-officedocument.presentationml.presentation")) {
// PowerPoint Document - OpenXML file format
- XMLSlideShow slideShow = new XMLSlideShow(pPart.getInputStream());
- slideShow.close();
+ try (InputStream stream = pPart.getInputStream()) {
+ XMLSlideShow slideShow = new XMLSlideShow(stream);
+ slideShow.close();
+ }
} else {
// Any other type of embedded object.
System.out.println("Unknown Embedded Document: " + contentType);
- InputStream inputStream = pPart.getInputStream();
- inputStream.close();
+ try (InputStream inputStream = pPart.getInputStream()) {
+
+ }
}
}
}
Modified: poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/AddVideoToPptx.java.txt
URL: http://svn.apache.org/viewvc/poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/AddVideoToPptx.java.txt?rev=1896472&r1=1896471&r2=1896472&view=diff
==============================================================================
--- poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/AddVideoToPptx.java.txt (original)
+++ poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/AddVideoToPptx.java.txt Tue Dec 28 10:51:24 2021
@@ -166,14 +166,16 @@ public class AddVideoToPptx {
ext.setUri("{DAA4B4D4-6D71-4841-9C94-3DE7FCFB9230}");
String p14Ns = "http://schemas.microsoft.com/office/powerpoint/2010/main";
XmlCursor cur = ext.newCursor();
- cur.toEndToken();
- cur.beginElement(new QName(p14Ns, "media", "p14"));
- cur.insertNamespace("p14", p14Ns);
- cur.insertAttributeWithValue(new QName(STRelationshipId.type.getName().getNamespaceURI(), "embed"), prsEmbed1.getId());
- cur.beginElement(new QName(p14Ns, "trim", "p14"));
- cur.insertAttributeWithValue("st", df_time.format(seconds*1000.0));
- cur.dispose();
-
+ try {
+ cur.toEndToken();
+ cur.beginElement(new QName(p14Ns, "media", "p14"));
+ cur.insertNamespace("p14", p14Ns);
+ cur.insertAttributeWithValue(new QName(STRelationshipId.type.getName().getNamespaceURI(), "embed"), prsEmbed1.getId());
+ cur.beginElement(new QName(p14Ns, "trim", "p14"));
+ cur.insertAttributeWithValue("st", df_time.format(seconds*1000.0));
+ } finally {
+ cur.dispose();
+ }
}
static void addTimingInfo(XSLFSlide slide1, XSLFPictureShape pic1) {
Modified: poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/LinkVideoToPptx.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/LinkVideoToPptx.java?rev=1896472&r1=1896471&r2=1896472&view=diff
==============================================================================
--- poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/LinkVideoToPptx.java (original)
+++ poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xslf/LinkVideoToPptx.java Tue Dec 28 10:51:24 2021
@@ -85,12 +85,14 @@ public final class LinkVideoToPptx {
String p14Ns = "http://schemas.microsoft.com/office/powerpoint/2010/main";
XmlCursor cur = ext.newCursor();
- cur.toEndToken();
- cur.beginElement(new QName(p14Ns, "media", "p14"));
- cur.insertNamespace("p14", p14Ns);
- cur.insertAttributeWithValue(new QName(CORE_PROPERTIES_ECMA376_NS, "link"), prsEmbed1.getId());
- cur.dispose();
-
+ try {
+ cur.toEndToken();
+ cur.beginElement(new QName(p14Ns, "media", "p14"));
+ cur.insertNamespace("p14", p14Ns);
+ cur.insertAttributeWithValue(new QName(CORE_PROPERTIES_ECMA376_NS, "link"), prsEmbed1.getId());
+ } finally {
+ cur.dispose();
+ }
CTSlide xslide = slide1.getXmlObject();
CTTimeNodeList ctnl;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org