baliuka     02/02/10 02:00:12

  Modified:    simplestore/src/java/org/apache/commons/simplestore
                        SoftRefMemoryStore.java Store.java
                        SynchronizedStore.java
               simplestore/src/sample/org/apache/commons/simplestore/persistence
                        PersistenceManager.java TransactionImpl.java
               simplestore/src/test/org/apache/commons/simplestore
                        TestSoftRefMemoryStore.java
  Removed:     simplestore/src/java/org/apache/commons/simplestore
                        Reachable.java Swap.java
  Log:
  Removed some unused code
  
  Revision  Changes    Path
  1.11      +13 -46    
jakarta-commons-sandbox/simplestore/src/java/org/apache/commons/simplestore/SoftRefMemoryStore.java
  
  Index: SoftRefMemoryStore.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/simplestore/src/java/org/apache/commons/simplestore/SoftRefMemoryStore.java,v
  retrieving revision 1.10
  retrieving revision 1.11
  diff -u -r1.10 -r1.11
  --- SoftRefMemoryStore.java   9 Feb 2002 18:45:11 -0000       1.10
  +++ SoftRefMemoryStore.java   10 Feb 2002 10:00:12 -0000      1.11
  @@ -68,7 +68,7 @@
    *      [EMAIL PROTECTED]</a>
    *@author     Gerhard Froehlich <a href="mailto:[EMAIL PROTECTED]";>
    *      [EMAIL PROTECTED]</a>
  - *@version    $Id: SoftRefMemoryStore.java,v 1.10 2002/02/09 18:45:11 froehlich Exp 
$
  + *@version    $Id: SoftRefMemoryStore.java,v 1.11 2002/02/10 10:00:12 baliuka Exp $
    */
   public class SoftRefMemoryStore
            implements Store {
  @@ -79,7 +79,7 @@
       private int m_current = 0;
       private Map m_map;
       private ReferenceQueue m_queue = new ReferenceQueue();
  -    private Swap m_swap;
  +   
   
       /**
        * Creates new SoftRefMemoryStore
  @@ -88,8 +88,8 @@
        *@param  swap               Description of Parameter
        *@param  maxStrongRefCount  Description of Parameter
        */
  -    protected SoftRefMemoryStore(Map map, Swap swap, int maxStrongRefCount) {
  -        this.m_swap = swap;
  +    protected SoftRefMemoryStore(Map map, int maxStrongRefCount) {
  +
           this.m_map = map;
   
           if (maxStrongRefCount < 0) {
  @@ -102,8 +102,8 @@
           }
       }
   
  -    public static Store getInstance(Map map, Swap swap, int maxStrongRef) {
  -        return new SynchronizedStore(new SoftRefMemoryStore(map, swap, 
maxStrongRef));
  +    public static Store getInstance(Map map,  int maxStrongRef) {
  +        return new SynchronizedStore(new SoftRefMemoryStore(map, maxStrongRef));
       }
   
       /**
  @@ -118,14 +118,9 @@
           SoftRef ref = (SoftRef) m_map.get(key);
   
           if (ref != null) {
  -            Object value = ref.get();
  -
  -            if (value != null && value instanceof StrongRef) {
  -                object = ((StrongRef) value).object;
  -            } else {
  -                object = value;
  -            }
  +            object = ref.get();
           }
  +        
           addStrongRef(object);
           return object;
       }
  @@ -178,19 +173,10 @@
           m_map.clear();
       }
   
  -    private SoftRef makeValue(Object key, Object value, ReferenceQueue queue, Swap 
swap) {
  -        if (swap == null || value == null) {
  +    private SoftRef makeValue(Object key, Object value, ReferenceQueue queue) {
  +        
               return new SoftRef(key, value, queue);
  -        } else {
  -            if (!(value instanceof Reachable)) {
  -                throw new java.lang.IllegalStateException("Value not Reachable in 
Swap ");
  -            }
  -            Reachable val = (Reachable) value;
  -            StrongRef strong = new StrongRef(key, val, swap);
  -            SoftRef ref = new SoftRef(key, strong, queue);
  -            val.setReference(strong);
  -            return ref;
  -        }
  +        
       }
   
       // remove unused keys
  @@ -214,7 +200,7 @@
       }
   
       private void internalStoreObject(Object key, Object object) {
  -        SoftRef ref = makeValue(key, object, m_queue, m_swap);
  +        SoftRef ref = makeValue(key, object, m_queue);
           addStrongRef(ref.get());
           m_map.put(key, ref);
       }
  @@ -228,25 +214,6 @@
           }
       }
   
  -    static class StrongRef extends Object {
  -        Reachable object;
  -        Object key;
  -        Swap queue;
  -
  -        private StrongRef(Object key, Reachable object, Swap queue) {
  -            this.queue = queue;// used in finalize
  -            this.object = object;// add strong reference to value
  -            this.key = key;
  -        }
  -
  -        public Object get() {
  -            return object;
  -        }
  -
  -        protected void finalize() throws Throwable {
  -            super.finalize();
  -            queue.add(key, object);
  -        }
  -    }
  +   
   }
   
  
  
  
  1.12      +1 -8      
jakarta-commons-sandbox/simplestore/src/java/org/apache/commons/simplestore/Store.java
  
  Index: Store.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/simplestore/src/java/org/apache/commons/simplestore/Store.java,v
  retrieving revision 1.11
  retrieving revision 1.12
  diff -u -r1.11 -r1.12
  --- Store.java        9 Feb 2002 18:45:11 -0000       1.11
  +++ Store.java        10 Feb 2002 10:00:12 -0000      1.12
  @@ -59,22 +59,15 @@
    *
    * @author Gerhard Froehlich <a href="mailto:[EMAIL PROTECTED]";>
    *      [EMAIL PROTECTED]</a>
  - * @version $Id: Store.java,v 1.11 2002/02/09 18:45:11 froehlich Exp $
  + * @version $Id: Store.java,v 1.12 2002/02/10 10:00:12 baliuka Exp $
    */
   public interface Store {
  -       
  -    void clear();
       
       Object get(Object key);
       
  -    boolean isEmpty();
  -    
       void put(Object key, Object value);
       
       Object remove(Object key);
             
  -    boolean containsKey(Object key);
  -    
  -    int size();
   }
   
  
  
  
  1.6       +2 -29     
jakarta-commons-sandbox/simplestore/src/java/org/apache/commons/simplestore/SynchronizedStore.java
  
  Index: SynchronizedStore.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/simplestore/src/java/org/apache/commons/simplestore/SynchronizedStore.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- SynchronizedStore.java    9 Feb 2002 18:45:11 -0000       1.5
  +++ SynchronizedStore.java    10 Feb 2002 10:00:12 -0000      1.6
  @@ -64,7 +64,7 @@
    *      [EMAIL PROTECTED]</a>
    *@author     Gerhard Froehlich <a href="mailto:[EMAIL PROTECTED]";>
    *      [EMAIL PROTECTED]</a>
  - *@version    $Id: SynchronizedStore.java,v 1.5 2002/02/09 18:45:11 froehlich Exp $
  + *@version    $Id: SynchronizedStore.java,v 1.6 2002/02/10 10:00:12 baliuka Exp $
    */
   final class SynchronizedStore
            implements Store {
  @@ -95,12 +95,7 @@
           }
       }
   
  -    public boolean isEmpty() {
  -        synchronized (store) {
  -            return store.isEmpty();
  -        }
  -    }
  -
  +   
       /**
        * Remove the object associated to the given key.
        *
  @@ -113,17 +108,6 @@
           }
       }
   
  -    /**
  -     * Indicates if the given key is associated to a contained object.
  -     *
  -     *@param  key  the Key Object
  -     *@return
  -     */
  -    public boolean containsKey(Object key) {
  -        synchronized (store) {
  -            return store.containsKey(key);
  -        }
  -    }
   
       public void put(Object key, Object value) {
           synchronized (store) {
  @@ -131,16 +115,5 @@
           }
       }
   
  -    public int size() {
  -        synchronized (store) {
  -            return store.size();
  -        }
  -    }
  -
  -    public void clear() {
  -        synchronized (store) {
  -            store.clear();
  -        }
  -    }
   }
   
  
  
  
  1.4       +2 -2      
jakarta-commons-sandbox/simplestore/src/sample/org/apache/commons/simplestore/persistence/PersistenceManager.java
  
  Index: PersistenceManager.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/simplestore/src/sample/org/apache/commons/simplestore/persistence/PersistenceManager.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- PersistenceManager.java   9 Feb 2002 18:51:43 -0000       1.3
  +++ PersistenceManager.java   10 Feb 2002 10:00:12 -0000      1.4
  @@ -59,7 +59,7 @@
   /**
    *@author     Juozas Baliuka <a href="mailto:[EMAIL PROTECTED]";>
    *      [EMAIL PROTECTED]</a>
  - *@version    $Id: PersistenceManager.java,v 1.3 2002/02/09 18:51:43 froehlich Exp $
  + *@version    $Id: PersistenceManager.java,v 1.4 2002/02/10 10:00:12 baliuka Exp $
    */
   
   public class PersistenceManager {
  @@ -70,7 +70,7 @@
   
       private Storage storage;
       private TransactionManager transactionManager;
  -    private Store store = SoftRefMemoryStore.getInstance(new java.util.HashMap(), 
null, 0xFF);
  +    private Store store = SoftRefMemoryStore.getInstance(new java.util.HashMap(),  
0xFF);
   
       /**
        * Creates new PersiatenceManager
  
  
  
  1.3       +2 -2      
jakarta-commons-sandbox/simplestore/src/sample/org/apache/commons/simplestore/persistence/TransactionImpl.java
  
  Index: TransactionImpl.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/simplestore/src/sample/org/apache/commons/simplestore/persistence/TransactionImpl.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- TransactionImpl.java      9 Feb 2002 18:51:43 -0000       1.2
  +++ TransactionImpl.java      10 Feb 2002 10:00:12 -0000      1.3
  @@ -58,12 +58,12 @@
   /**
    *@author     Juozas Baliuka <a href="mailto:[EMAIL PROTECTED]";>
    *      [EMAIL PROTECTED]</a>
  - *@version    $Id: TransactionImpl.java,v 1.2 2002/02/09 18:51:43 froehlich Exp $
  + *@version    $Id: TransactionImpl.java,v 1.3 2002/02/10 10:00:12 baliuka Exp $
    */
   
   public class TransactionImpl implements InternalTransaction {
   
  -    private static Store instances = SoftRefMemoryStore.getInstance(new 
java.util.HashMap(), null, 0x00);
  +    private static Store instances = SoftRefMemoryStore.getInstance(new 
java.util.HashMap(), 0x00);
   
       int threadId = getCurrentThreadId();
       InternalTransaction transaction;
  
  
  
  1.10      +7 -46     
jakarta-commons-sandbox/simplestore/src/test/org/apache/commons/simplestore/TestSoftRefMemoryStore.java
  
  Index: TestSoftRefMemoryStore.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons-sandbox/simplestore/src/test/org/apache/commons/simplestore/TestSoftRefMemoryStore.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -r1.9 -r1.10
  --- TestSoftRefMemoryStore.java       9 Feb 2002 19:19:01 -0000       1.9
  +++ TestSoftRefMemoryStore.java       10 Feb 2002 10:00:12 -0000      1.10
  @@ -65,7 +65,7 @@
    *      [EMAIL PROTECTED]</a>
    *@author     Juozas Baliuka <a href="mailto:[EMAIL PROTECTED]";>
    *      [EMAIL PROTECTED]</a>
  - *@version    $Id: TestSoftRefMemoryStore.java,v 1.9 2002/02/09 19:19:01 froehlich 
Exp $
  + *@version    $Id: TestSoftRefMemoryStore.java,v 1.10 2002/02/10 10:00:12 baliuka 
Exp $
    */
   public class TestSoftRefMemoryStore extends TestStore {
   
  @@ -88,27 +88,11 @@
       }
   
       public Store makeStore() {
  -        Swap swap =
  -            new Swap() {
  -                // this means GC decided memory is low and
  -                // SoftRefMemoryStore decided object is "old"
  -                // value wrapper calls this method from finalize
  -                // Queue implementation can write object to swap here
  -                // It can use JMS to send Object Message
  -                public void add(Object key, Reachable object) {
  -                    System.out.println(key + " Object added to swap " + 
object.toString().substring(0, 20));
  -                }
  -                // do nothing at this time, becouse we added no object to Qeue at 
this time
  -                // Queue implementation can add objects to Batch, then this method 
can be usd to test Queue
  -                // It can use Thread in stand alone application to execute Batch
  -
  -
  -            };
  -        SoftRefMemoryStore store = new SoftRefMemoryStore(new HashMap(ITERATIONS), 
swap, MAX_STRONG_REFS);
  +        SoftRefMemoryStore store = new SoftRefMemoryStore(new HashMap(ITERATIONS),  
MAX_STRONG_REFS);
           return store;
       }
   
  -    public void testSwaping() {
  +    public void testStore() {
   
           assertTrue("try to set ITERATIONS > MAX_STRONG_REFS ", ITERATIONS > 
MAX_STRONG_REFS);
   
  @@ -116,12 +100,9 @@
           java.util.Arrays.fill(testObject, '.');
           Object key = new Integer(0);
           Object value = new String(testObject);
  -        ValueObject valueObj = new ValueObject(value);
  -        store.put(key, valueObj);
  +        store.put(key, value);
           for (int i = 1; i < ITERATIONS; i++) {
  -            Object obj = new String(testObject);
  -            obj = new ValueObject(obj);
  -            store.put(new Integer(i), obj);
  +            store.put(new Integer(i), new String(testObject));
               _testGet(i);
           }
           // get something
  @@ -129,7 +110,7 @@
                   i < ITERATIONS / 2; i++) {
               _testGet(i);
           }
  -        assertEquals("Lost Strong Referense in cache", store.get(key), valueObj);
  +        assertEquals("Lost Strong Referense in cache", store.get(key), value);
   
       }
   
  @@ -160,26 +141,6 @@
           }
       }
   
  -    public static class ValueObject implements Reachable {
  -        Object reference;
  -        Object value;
  -
  -        public ValueObject(Object value) {
  -            this.value = value;
  -        }
  -
  -        public void setReference(Object reference) {
  -            this.reference = reference;
  -        }
  -
  -        public String toString() {
  -            return value != null ? value.toString() : null;
  -        }
  -
  -        Object getValue() {
  -            return value;
  -        }
  -
  -    }
  +    
   }
   
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to