Author: andy
Date: Tue Apr 16 14:23:38 2013
New Revision: 1468428
URL: http://svn.apache.org/r1468428
Log:
Remove potential concurrency problem.
Modified:
jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Lib.java
Modified: jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Lib.java
URL:
http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Lib.java?rev=1468428&r1=1468427&r2=1468428&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Lib.java
(original)
+++ jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Lib.java Tue
Apr 16 14:23:38 2013
@@ -91,13 +91,12 @@ public class Lib
catch (InterruptedException ex) { Log.warn(Lib.class, "interrupted",
ex) ; }
}
- // Confess - these constructors are very cheap (objects are one private
int) so this is unnecessary.
- private static Checksum crc32 = new CRC32() ;
- private static Checksum adler32 = new Adler32() ;
-
+ /**
+ * @see CRC32
+ */
public static long crc32(byte[] bytes)
{
- return crc(crc32, bytes) ;
+ return crc(new CRC32(), bytes) ;
}
/** Faster than CRC32, nearly as good.
@@ -105,7 +104,7 @@ public class Lib
*/
public static long adler32(byte[] bytes)
{
- return crc(adler32, bytes) ;
+ return crc(new Adler32(), bytes) ;
}
private static long crc(Checksum alg, byte[] bytes)
