/*
 *  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.tomcat.util.bcel.classfile;

import java.io.DataInputStream;
import java.io.IOException;
import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.Map;

import org.apache.tomcat.util.bcel.Constants;

/**
 * This class is derived from the abstract
 * <A HREF="org.apache.tomcat.util.bcel.classfile.Constant.html">Constant</A> class
 * and represents a reference to a Utf8 encoded string.
 *
 * @author  <A HREF="mailto:m.dahm@gmx.de">M. Dahm</A>
 * @see     Constant
 */
public final class ConstantUtf8 extends Constant {

    private static final long serialVersionUID = 8119001312020421976L;
    private final String bytes;

    private static final int MAX_CACHE_ENTRIES = 20000;
    private static final int INITIAL_CACHE_CAPACITY = (int)(MAX_CACHE_ENTRIES/0.75);
    private static HashMap<String, ConstantUtf8> cache;

    private static synchronized ConstantUtf8 getCachedInstance(String s) {
        if (s.length() > 200) {
            return  new ConstantUtf8(s);
        }
        if (cache == null)  {
            cache = new LinkedHashMap<String, ConstantUtf8>(INITIAL_CACHE_CAPACITY, 0.75f, true) {
                private static final long serialVersionUID = 1L;

                @Override
                protected boolean removeEldestEntry(Map.Entry<String, ConstantUtf8> eldest) {
                     return size() > MAX_CACHE_ENTRIES;
                }
            };
        }
        ConstantUtf8 result = cache.get(s);
        if (result != null) {
                return result;
            }
        result = new ConstantUtf8(s);
        cache.put(s, result);
        return result;
    }

    private static ConstantUtf8 getInstance(String s) {
        return getCachedInstance(s);
    }

    static ConstantUtf8 getInstance(DataInputStream file) throws IOException {
        return getInstance(file.readUTF());
    }


    /**
     * @param bytes Data
     */
    private ConstantUtf8(String bytes) {
        super(Constants.CONSTANT_Utf8);
        if (bytes == null) {
            throw new IllegalArgumentException("bytes must not be null!");
        }
        this.bytes = bytes;
    }


    /**
     * @return Data converted to string.
     */
    public final String getBytes() {
        return bytes;
    }
}

@@ -918,88 +918,8 @@
 ion;
-%0Aimport java.util.HashMap;%0Aimport java.util.LinkedHashMap;%0Aimport java.util.Map;
 %0A%0Aim
@@ -1393,1094 +1393,8 @@
 s;%0A%0A
-    private static final int MAX_CACHE_ENTRIES = 20000;%0A    private static final int INITIAL_CACHE_CAPACITY = (int)(MAX_CACHE_ENTRIES/0.75);%0A    private static HashMap%3CString, ConstantUtf8%3E cache;%0A%0A    private static synchronized ConstantUtf8 getCachedInstance(String s) %7B%0A        if (s.length() %3E 200) %7B%0A            return  new ConstantUtf8(s);%0A        %7D%0A        if (cache == null)  %7B%0A            cache = new LinkedHashMap%3CString, ConstantUtf8%3E(INITIAL_CACHE_CAPACITY, 0.75f, true) %7B%0A                private static final long serialVersionUID = 1L;%0A%0A                @Override%0A                protected boolean removeEldestEntry(Map.Entry%3CString, ConstantUtf8%3E eldest) %7B%0A                     return size() %3E MAX_CACHE_ENTRIES;%0A                %7D%0A            %7D;%0A        %7D%0A        ConstantUtf8 result = cache.get(s);%0A        if (result != null) %7B%0A                return result;%0A            %7D%0A        result = new ConstantUtf8(s);%0A        cache.put(s, result);%0A        return result;%0A    %7D%0A%0A    private static ConstantUtf8 getInstance(String s) %7B%0A        return getCachedInstance(s);%0A    %7D%0A
 %0A   
@@ -1484,27 +1484,32 @@
  return 
-getInstance
+new ConstantUtf8
 (file.re

Remove the cache. Generally, it makes performance worse not better. Testing with the jars that ship with Jira, performance was improved by around 10-15% by removing the cache. Based on a suggestion by hzhang9