You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by je...@apache.org on 2010/10/22 13:49:47 UTC

svn commit: r1026306 - /pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/util/TestLayerUtility.java

Author: jeremias
Date: Fri Oct 22 11:49:47 2010
New Revision: 1026306

URL: http://svn.apache.org/viewvc?rev=1026306&view=rev
Log:
Forgot to close a PDDocument.

Modified:
    pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/util/TestLayerUtility.java

Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/util/TestLayerUtility.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/util/TestLayerUtility.java?rev=1026306&r1=1026305&r2=1026306&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/util/TestLayerUtility.java (original)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/util/TestLayerUtility.java Fri Oct 22 11:49:47 2010
@@ -91,24 +91,30 @@ public class TestLayerUtility extends Te
         }
 
         PDDocument doc = PDDocument.load(targetFile);
-        PDDocumentCatalog catalog = doc.getDocumentCatalog();
-
-        //OCGs require PDF 1.5 or later
-        //TODO need some comfortable way to enable/check the PDF version
-        //assertEquals("%PDF-1.5", doc.getDocument().getHeaderString());
-        //assertEquals("1.5", catalog.getVersion());
-
-        PDPage page = (PDPage)catalog.getAllPages().get(0);
-        PDPropertyList props = page.findResources().getProperties();
-        assertNotNull(props);
-        PDOptionalContentGroup ocg = props.getOptionalContentGroup(COSName.getPDFName("MC0"));
-        assertNotNull(ocg);
-        assertEquals("overlay", ocg.getName());
-
-        PDOptionalContentProperties ocgs = catalog.getOCProperties();
-        PDOptionalContentGroup overlay = ocgs.getGroup("overlay");
-        assertEquals(ocg.getName(), overlay.getName());
+        try
+        {
+            PDDocumentCatalog catalog = doc.getDocumentCatalog();
 
+            //OCGs require PDF 1.5 or later
+            //TODO need some comfortable way to enable/check the PDF version
+            //assertEquals("%PDF-1.5", doc.getDocument().getHeaderString());
+            //assertEquals("1.5", catalog.getVersion());
+
+            PDPage page = (PDPage)catalog.getAllPages().get(0);
+            PDPropertyList props = page.findResources().getProperties();
+            assertNotNull(props);
+            PDOptionalContentGroup ocg = props.getOptionalContentGroup(COSName.getPDFName("MC0"));
+            assertNotNull(ocg);
+            assertEquals("overlay", ocg.getName());
+
+            PDOptionalContentProperties ocgs = catalog.getOCProperties();
+            PDOptionalContentGroup overlay = ocgs.getGroup("overlay");
+            assertEquals(ocg.getName(), overlay.getName());
+        }
+        finally
+        {
+            doc.close();
+        }
     }
 
     private File createMainPDF() throws IOException, COSVisitorException