Author: acumiskey
Date: Mon Jul 21 05:58:18 2008
New Revision: 678405
URL: http://svn.apache.org/viewvc?rev=678405&view=rev
Log:
Must synchronize on cacheMap when removing DataObjectCache instance.
Modified:
xmlgraphics/fop/branches/Temp_AFPGOCAResources/src/java/org/apache/fop/render/afp/DataObjectCache.java
Modified:
xmlgraphics/fop/branches/Temp_AFPGOCAResources/src/java/org/apache/fop/render/afp/DataObjectCache.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_AFPGOCAResources/src/java/org/apache/fop/render/afp/DataObjectCache.java?rev=678405&r1=678404&r2=678405&view=diff
==============================================================================
---
xmlgraphics/fop/branches/Temp_AFPGOCAResources/src/java/org/apache/fop/render/afp/DataObjectCache.java
(original)
+++
xmlgraphics/fop/branches/Temp_AFPGOCAResources/src/java/org/apache/fop/render/afp/DataObjectCache.java
Mon Jul 21 05:58:18 2008
@@ -25,6 +25,7 @@
import java.io.RandomAccessFile;
import java.nio.MappedByteBuffer;
import java.nio.channels.FileChannel;
+import java.util.Collections;
import java.util.Map;
import org.apache.commons.logging.Log;
@@ -48,6 +49,7 @@
private static Map/*<Integer,DataObjectCache>*/ cacheMap
= new java.util.HashMap/*<Integer,DataObjectCache>*/();
+
/** Mapping of data object uri --> cache record */
private Map/*<ResourceInfo,Record>*/ includableMap
= new java.util.HashMap/*<ResourceInfo,Record>*/();
@@ -110,7 +112,9 @@
try {
raFile.close();
tempFile.delete();
- cacheMap.remove(id); // remove ourselves from the cache map
+ synchronized (cacheMap) {
+ cacheMap.remove(id); // remove ourselves from the cache map
+ }
} catch (IOException e) {
log.error("Failed to close temporary file");
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
