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 2020/06/13 11:45:40 UTC

svn commit: r1878804 - /pdfbox/branches/issue45/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java

Author: tilman
Date: Sat Jun 13 11:45:40 2020
New Revision: 1878804

URL: http://svn.apache.org/viewvc?rev=1878804&view=rev
Log:
PDFBOX-4875: synchronize on FONTS instead of on parameter

Modified:
    pdfbox/branches/issue45/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java

Modified: pdfbox/branches/issue45/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue45/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java?rev=1878804&r1=1878803&r2=1878804&view=diff
==============================================================================
--- pdfbox/branches/issue45/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java (original)
+++ pdfbox/branches/issue45/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java Sat Jun 13 11:45:40 2020
@@ -172,9 +172,7 @@ final class Standard14Fonts
 
         if (FONTS.get(baseName) == null)
         {
-            // baseName is a reference to a string that was statically
-            // put in the ALIASES map so we can safely synchronize on it
-            synchronized (baseName)
+            synchronized (FONTS)
             {
                 if (FONTS.get(baseName) == null)
                 {