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;
   }
 }