You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2009/07/01 21:55:15 UTC

svn commit: r790347 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/util/StringMap.java

Author: rickhall
Date: Wed Jul  1 19:55:15 2009
New Revision: 790347

URL: http://svn.apache.org/viewvc?rev=790347&view=rev
Log:
Optimization for StringMap. (FELIX-1293)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/util/StringMap.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/util/StringMap.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/util/StringMap.java?rev=790347&r1=790346&r2=790347&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/util/StringMap.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/util/StringMap.java Wed Jul  1 19:55:15 2009
@@ -1,4 +1,4 @@
-/* 
+/*
  * 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
@@ -36,12 +36,12 @@
     {
         this(true);
     }
-    
+
     public StringMap(boolean caseSensitive)
     {
         m_map = new TreeMap(new StringComparator(caseSensitive));
     }
-    
+
     public StringMap(Map map, boolean caseSensitive)
     {
         this(caseSensitive);
@@ -55,9 +55,12 @@
 
     public void setCaseSensitive(boolean b)
     {
-        TreeMap map = new TreeMap(new StringComparator(b));
-        map.putAll(m_map);
-        m_map = map;
+        if (isCaseSensitive() != b)
+        {
+            TreeMap map = new TreeMap(new StringComparator(b));
+            map.putAll(m_map);
+            m_map = map;
+        }
     }
 
     public int size()