Some code hygeine from Charlie.

src/java/com/samskivert/depot/DepotRepository.java

@@ -368,6 +368,9 @@ public abstract class DepotRepository
                 throw new IllegalArgumentException(
                     "Cannot use " + cache + " strategy because " + reason);
             }
+            break;
+        case NONE: case CONTENTS:
+            break;// NONE and CONTENTS can always be used.
         }
 
         if (!_ctx.isUsingCache()) {

src/java/com/samskivert/depot/EHCacheAdapter.java

@@ -353,7 +353,7 @@ public class EHCacheAdapter
     }
 
     protected CacheEventListener _cacheEventListener = new CacheEventListener() {
-        public Object clone () throws CloneNotSupportedException {
+        @Override public Object clone () throws CloneNotSupportedException {
             throw new CloneNotSupportedException();
         }
         public void dispose () {}

src/java/com/samskivert/depot/KeySet.java

@@ -51,7 +51,7 @@ import com.samskivert.depot.operator.Or;
  * the cache.
  */
 public abstract class KeySet<T extends PersistentRecord> extends WhereClause
-    implements Serializable, SQLExpression, ValidatingCacheInvalidator, Iterable<Key<T>>
+    implements Serializable, ValidatingCacheInvalidator, Iterable<Key<T>>
 {
     /**
      * Creates a key set for the supplied persistent record and keys.