ResourceBundle.java (BundleKey): Don't implement Cloneable.
authorBryce McKinlay <mckinlay@redhat.com>
Thu, 25 Nov 2004 19:20:17 +0000 (19:20 +0000)
committerBryce McKinlay <bryce@gcc.gnu.org>
Thu, 25 Nov 2004 19:20:17 +0000 (19:20 +0000)
2004-11-25  Bryce McKinlay  <mckinlay@redhat.com>

* java/util/ResourceBundle.java (BundleKey): Don't implement
Cloneable.
(BundleKey.clone): Removed.
(getBundle): Copy BundleKey using constructor, not clone().

From-SVN: r91304

libjava/ChangeLog
libjava/java/util/ResourceBundle.java

index 1de7a72..1f26c2b 100644 (file)
@@ -1,3 +1,10 @@
+2004-11-25  Bryce McKinlay  <mckinlay@redhat.com>
+
+       * java/util/ResourceBundle.java (BundleKey): Don't implement
+       Cloneable.
+       (BundleKey.clone): Removed.
+       (getBundle): Copy BundleKey using constructor, not clone().
+
 2004-11-25  Kelley Cook  <kcook@gcc.gnu.org>
 
        * configure: Regenerate for libtool reversion.
index 94acc96..57a9bfc 100644 (file)
@@ -244,7 +244,7 @@ public abstract class ResourceBundle
 
   /** Cache key for the ResourceBundle cache.  Resource bundles are keyed
       by the combination of bundle name, locale, and class loader. */
-  private static class BundleKey implements Cloneable
+  private static class BundleKey
   {
     String baseName;
     Locale locale;
@@ -281,18 +281,6 @@ public abstract class ResourceBundle
        baseName.equals(key.baseName) &&
         locale.equals(key.locale) &&
        classLoader.equals(key.classLoader);
-    }
-    
-    public Object clone()
-    {
-      Object clone = null;
-      try
-      {
-       clone = super.clone();
-      }
-      catch (CloneNotSupportedException x) {}
-      
-      return clone;
     }    
   }
   
@@ -417,7 +405,7 @@ public abstract class ResourceBundle
        if (bundle == null && !locale.equals(defaultLocale))
          bundle = tryBundle(baseName, defaultLocale, classLoader, true);
 
-       BundleKey key = (BundleKey) lookupKey.clone();
+       BundleKey key = new BundleKey(baseName, locale, classLoader);
         if (bundle == null)
          {
            // Cache the fact that this lookup has previously failed.