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 2019/08/07 18:06:20 UTC

svn commit: r1864661 - /pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java

Author: tilman
Date: Wed Aug  7 18:06:20 2019
New Revision: 1864661

URL: http://svn.apache.org/viewvc?rev=1864661&view=rev
Log:
PDFBOX-4071: simplify code

Modified:
    pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java

Modified: pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java?rev=1864661&r1=1864660&r2=1864661&view=diff
==============================================================================
--- pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java (original)
+++ pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java Wed Aug  7 18:06:20 2019
@@ -369,11 +369,9 @@ final class FileSystemFontProvider exten
      */
     private void saveDiskCache()
     {
-        File file = null;
         try
         {
-            file = getDiskCacheFile();
-
+            File file = getDiskCacheFile();
             try (BufferedWriter writer = new BufferedWriter(new FileWriter(file)))
             {
                 for (FSFontInfo fontInfo : fontInfoList)
@@ -387,12 +385,10 @@ final class FileSystemFontProvider exten
                 LOG.warn("Installed fonts information will have to be reloaded for each start");
                 LOG.warn("You can assign a directory to the 'pdfbox.fontcache' property");
             }
-
         }
         catch (SecurityException e)
         {
             LOG.debug("Couldn't create writer for font cache file", e);
-            return;
         }
     }