You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2010/07/29 15:06:14 UTC
svn commit: r980433 -
/lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/CodecProvider.java
Author: mikemccand
Date: Thu Jul 29 13:06:14 2010
New Revision: 980433
URL: http://svn.apache.org/viewvc?rev=980433&view=rev
Log:
add some missing synchronization to CodecProvider
Modified:
lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/CodecProvider.java
Modified: lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/CodecProvider.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/CodecProvider.java?rev=980433&r1=980432&r2=980433&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/CodecProvider.java (original)
+++ lucene/dev/trunk/lucene/src/java/org/apache/lucene/index/codecs/CodecProvider.java Thu Jul 29 13:06:14 2010
@@ -49,7 +49,7 @@ public abstract class CodecProvider {
public final static String[] CORE_CODECS = new String[] {"Standard", "Sep", "Pulsing", "IntBlock", "PreFlex"};
- public void register(Codec codec) {
+ public synchronized void register(Codec codec) {
if (codec.name == null) {
throw new IllegalArgumentException("code.name is null");
}
@@ -62,7 +62,7 @@ public abstract class CodecProvider {
}
/** @lucene.internal */
- public void unregister(Codec codec) {
+ public synchronized void unregister(Codec codec) {
if (codec.name == null) {
throw new IllegalArgumentException("code.name is null");
}
@@ -80,7 +80,7 @@ public abstract class CodecProvider {
return knownExtensions;
}
- public Codec lookup(String name) {
+ public synchronized Codec lookup(String name) {
final Codec codec = (Codec) codecs.get(name);
if (codec == null)
throw new IllegalArgumentException("required codec '" + name + "' not found");
@@ -104,11 +104,11 @@ public abstract class CodecProvider {
}
/** Used for testing. @lucene.internal */
- public static void setDefaultCodec(String s) {
+ public synchronized static void setDefaultCodec(String s) {
defaultCodec = s;
}
/** Used for testing. @lucene.internal */
- public static String getDefaultCodec() {
+ public synchronized static String getDefaultCodec() {
return defaultCodec;
}
}