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 2023/12/05 08:20:14 UTC

svn commit: r1914351 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java

Author: tilman
Date: Tue Dec  5 08:20:14 2023
New Revision: 1914351

URL: http://svn.apache.org/viewvc?rev=1914351&view=rev
Log:
PDFBOX-5727: use faster checksum as suggested by Gili Tzabari; add failed ttc font to cache

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

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java?rev=1914351&r1=1914350&r2=1914351&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java Tue Dec  5 08:20:14 2023
@@ -649,6 +649,7 @@ final class FileSystemFontProvider exten
         catch (IOException e)
         {
             LOG.warn("Could not load font file: {}", ttcFile, e);
+            fontInfoList.add(createFSIgnored(ttcFile, FontFormat.TTF, "*skipexception*"));
         }
     }
 
@@ -853,9 +854,9 @@ final class FileSystemFontProvider exten
         MessageDigest md;
         try
         {
-            md = MessageDigest.getInstance("SHA512");
-            byte[] md5 = md.digest(ba);
-            return Hex.getString(md5);
+            md = MessageDigest.getInstance("CRC32");
+            byte[] dig = md.digest(ba);
+            return Hex.getString(dig);
         }
         catch (NoSuchAlgorithmException ex)
         {