You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@abdera.apache.org by jm...@apache.org on 2007/04/25 02:09:18 UTC

svn commit: r532150 - /incubator/abdera/java/trunk/core/src/main/java/org/apache/abdera/factory/ExtensionFactoryMap.java

Author: jmsnell
Date: Tue Apr 24 17:09:17 2007
New Revision: 532150

URL: http://svn.apache.org/viewvc?view=rev&rev=532150
Log:
Synchronize puts to the extension wrapper hashmap

Modified:
    incubator/abdera/java/trunk/core/src/main/java/org/apache/abdera/factory/ExtensionFactoryMap.java

Modified: incubator/abdera/java/trunk/core/src/main/java/org/apache/abdera/factory/ExtensionFactoryMap.java
URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/core/src/main/java/org/apache/abdera/factory/ExtensionFactoryMap.java?view=diff&rev=532150&r1=532149&r2=532150
==============================================================================
--- incubator/abdera/java/trunk/core/src/main/java/org/apache/abdera/factory/ExtensionFactoryMap.java (original)
+++ incubator/abdera/java/trunk/core/src/main/java/org/apache/abdera/factory/ExtensionFactoryMap.java Tue Apr 24 17:09:17 2007
@@ -46,7 +46,7 @@
       for (ExtensionFactory factory : factories) {
         t = (T) factory.getElementWrapper(internal);
         if (t != null && t != internal) {
-          wrappers.put(internal, t);
+          setElementWrapper(internal,t);
           return t;
         }
       }
@@ -55,7 +55,7 @@
     return (t != null) ? t : (T)internal;
   }
   
-  public void setElementWrapper(Element internal, Element wrapper) {
+  public synchronized void setElementWrapper(Element internal, Element wrapper) {
     wrappers.put(internal, wrapper);
   }