You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2022/08/06 04:26:59 UTC

svn commit: r1903249 - /pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/font/PDFontTest.java

Author: tilman
Date: Sat Aug  6 04:26:59 2022
New Revision: 1903249

URL: http://svn.apache.org/viewvc?rev=1903249&view=rev
Log:
PDFBOX-4892: close doc, as suggested by Valery Bokov

Modified:
    pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/font/PDFontTest.java

Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/font/PDFontTest.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/font/PDFontTest.java?rev=1903249&r1=1903248&r2=1903249&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/font/PDFontTest.java (original)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/font/PDFontTest.java Sat Aug  6 04:26:59 2022
@@ -438,11 +438,13 @@ class PDFontTest
     {
         File fontFile = new File("target/fonts", "PDFBOX-5484.ttf");
         TrueTypeFont ttf = new TTFParser().parse(new RandomAccessReadBufferedFile(fontFile));
-        PDDocument doc = new PDDocument();
-        PDTrueTypeFont tr = PDTrueTypeFont.load(doc, ttf, WinAnsiEncoding.INSTANCE);
-        GeneralPath path1 = tr.getPath("oslash");
-        GeneralPath path2 = tr.getPath(248);
-        assertFalse(path2.getPathIterator(null).isDone()); // not empty
-        assertTrue(new Area(path1).equals(new Area(path2))); // assertEquals does not test equals()
+        try (PDDocument doc = new PDDocument())
+        {
+            PDTrueTypeFont tr = PDTrueTypeFont.load(doc, ttf, WinAnsiEncoding.INSTANCE);
+            GeneralPath path1 = tr.getPath("oslash");
+            GeneralPath path2 = tr.getPath(248);
+            assertFalse(path2.getPathIterator(null).isDone()); // not empty
+            assertTrue(new Area(path1).equals(new Area(path2))); // assertEquals does not test equals()
+        }
     }
 }