pdfbox-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From til...@apache.org
Subject svn commit: r1687904 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FontMapper.java
Date Sat, 27 Jun 2015 10:28:31 GMT
Author: tilman
Date: Sat Jun 27 10:28:31 2015
New Revision: 1687904

URL: http://svn.apache.org/r1687904
Log:
PDFBOX-2576: follow JLS conventions

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

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FontMapper.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FontMapper.java?rev=1687904&r1=1687903&r2=1687904&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FontMapper.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FontMapper.java Sat Jun
27 10:28:31 2015
@@ -46,7 +46,7 @@ final class FontMapper
     private FontMapper() {}
 
     private static final FontCache fontCache = new FontCache(); // todo: static cache isn't
ideal
-    private static final Log log = LogFactory.getLog(FontMapper.class);
+    private static final Log LOG = LogFactory.getLog(FontMapper.class);
     private static FontProvider fontProvider;
     private static Map<String, FontInfo> fontInfoByName;
 
@@ -81,7 +81,7 @@ final class FontMapper
     /**
      * Sets the font service provider.
      */
-    public synchronized static void setProvider(FontProvider fontProvider)
+    public static synchronized void setProvider(FontProvider fontProvider)
     {
         FontMapper.fontProvider = fontProvider;
         fontInfoByName = createFontInfoByName(fontProvider.getFontInfo());
@@ -90,7 +90,7 @@ final class FontMapper
     /**
      * Returns the font service provider. Defaults to using FileSystemFontProvider.
      */
-    public synchronized static FontProvider getProvider()
+    public static synchronized FontProvider getProvider()
     {
         if (fontProvider == null)
         {
@@ -315,7 +315,7 @@ final class FontMapper
             if (ttf == null)
             {
                 // we have to return something here as TTFs aren't strictly required on the
system
-                log.error("Using last-resort fallback for TTF font '" + fontName + "'");
+                LOG.error("Using last-resort fallback for TTF font '" + fontName + "'");
                 ttf = lastResortFont;
             }
             return new FontMapping<TrueTypeFont>(ttf, true);
@@ -343,7 +343,7 @@ final class FontMapper
             if (font == null)
             {
                 // we have to return something here as TTFs aren't strictly required on the
system
-                log.error("Using last-resort fallback for font '" + fontName + "'");
+                LOG.error("Using last-resort fallback for font '" + fontName + "'");
                 font = lastResortFont;
             }
             return new FontMapping<FontBoxFont>(font, true);
@@ -434,7 +434,7 @@ final class FontMapper
         // strip subset tag (happens when we substitute a corrupt embedded font, see PDFBOX-2642)
         if (postScriptName.contains("+"))
         {
-            postScriptName = postScriptName.substring(postScriptName.indexOf("+") + 1);
+            postScriptName = postScriptName.substring(postScriptName.indexOf('+') + 1);
         }
         
         // look up the PostScript name



Mime
View raw message