/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.apache.pdfbox.pdmodel.font;

import org.apache.fontbox.cmap.CMap;
import org.apache.fontbox.cmap.CMapParser;

import java.io.IOException;
import java.io.InputStream;
import java.util.Collections;
import java.util.HashMap;
import java.util.Map;

/**
 * CMap resource loader and cache.
 */
final class CMapManager
{
    static final Map<String, CMap> cMapCache =
            Collections.synchronizedMap(new HashMap<String, CMap>());

    private CMapManager()
    {
    }

    /**
     * Fetches the predefined CMap from disk (or cache).
     *
     * @param cMapName CMap name
     * @return The predefined CMap, never null.
     * @throws IOException 
     */
    public static CMap getPredefinedCMap(String cMapName) throws IOException
    {
        CMap cmap = cMapCache.get(cMapName);
        if (cmap != null)
        {
            return cmap;
        }

        CMap targetCmap = new CMapParser().parsePredefined(cMapName);

        // limit the cache to predefined CMaps
        cMapCache.put(targetCmap.getName(), targetCmap);
        return targetCmap;
    }

    /**
     * Parse the given CMap.
     *
     * @param cMapStream the CMap to be read
     * @return the parsed CMap
     */
    public static CMap parseCMap(InputStream cMapStream) throws IOException
    {
        CMap targetCmap = null;
        if (cMapStream != null)
        {
            // parse CMap using strict mode
            targetCmap = new CMapParser(true).parse(cMapStream);
        }
        return targetCmap;
    }
}

@@ -997,16 +997,18 @@
 tream;%0D%0A
+%0D%0A
 import j
@@ -1016,27 +1016,19 @@
 va.util.
-Collections
+Map
 ;%0D%0Aimpor
@@ -1043,35 +1043,33 @@
 til.
-HashMap;%0D%0Aimport java.util.
+concurrent.ConcurrentHash
 Map;
@@ -1157,16 +1157,24 @@
  static 
+private 
 final Ma
@@ -1193,89 +1193,46 @@
 ap%3E 
-cMapCache =%0D%0A            Collections.synchronizedMap(new HashMap%3CString, CMap
+CMAP_CACHE = new ConcurrentHashMap%3C
 %3E()
-)
 ;%0D%0A%0D
@@ -1576,25 +1576,26 @@
  cmap = 
-cMapCache
+CMAP_CACHE
 .get(cMa
@@ -1813,17 +1813,18 @@
     
-cMapCache
+CMAP_CACHE
 .put

optimize / simplify, as suggested by Valery Bokov