[jboss-cvs] JBossCache/src/org/jboss/cache ...

[Manik Surtani]

  User: msurtani
  Date: 06/08/30 15:09:56

  Modified:    src/org/jboss/cache    NodeImpl.java Region.java
                        TreeCache.java
  Log:
  removed unnenessary (IDE-generated) TODOs that were becoming noisy
  refactored identity lock constructors
  
  Revision  Changes    Path
  1.9       +1 -1      JBossCache/src/org/jboss/cache/NodeImpl.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: NodeImpl.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/NodeImpl.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -b -r1.8 -r1.9
  --- NodeImpl.java	30 Aug 2006 17:08:19 -0000	1.8
  +++ NodeImpl.java	30 Aug 2006 19:09:56 -0000	1.9
  @@ -141,7 +141,7 @@
      {
         if (lock_ == null)
         {
  -         lock_ = new IdentityLock(cache, fqn);
  +         lock_ = new IdentityLock(cache.getConfiguration().getIsolationLevel());
         }
      }
   
  
  
  
  1.3       +39 -42    JBossCache/src/org/jboss/cache/Region.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Region.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/Region.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -b -r1.2 -r1.3
  --- Region.java	18 Jul 2006 10:50:46 -0000	1.2
  +++ Region.java	30 Aug 2006 19:09:56 -0000	1.3
  @@ -6,7 +6,6 @@
    */
   package org.jboss.cache;
   
  -import org.w3c.dom.Element;
   import org.jboss.cache.eviction.EvictionPolicyConfig;
   import org.jboss.cache.marshall.RegionNameConflictException;
   import org.jboss.cache.marshall.RegionNotFoundException;
  @@ -20,26 +19,23 @@
   {
       /**
        * Registers a specific {@link ClassLoader} (rather than the default) for a region, represented by a {@link Fqn}.
  -     * @param fqn
  +    *
        * @param classLoader
        */
       void registerContextClassLoader(ClassLoader classLoader) throws RegionNameConflictException;
   
       /**
        * Unregisters any specific {@link ClassLoader}s from a region.
  -     * @param fqn
        */
       void unregisterContextClassLoader() throws RegionNotFoundException;
   
       /**
        * Activates a region for replication (by default, the entire cache is activated)
  -     * @param fqn
        */
       void activate() throws RegionNameConflictException;
   
       /**
        * Deactivates a region from being replicated.
  -     * @param fqn
        */
       void deactivate() throws RegionNameConflictException;
   
  @@ -47,13 +43,14 @@
   
       /**
        * Looks up a {@link ClassLoader} for a given region.
  -     * @param fqn
  +    *
        * @return a ClassLoader
        */
       ClassLoader getClassLoader();
   
       /**
        * Configures an eviction policy for the current region.
  +    *
        * @param evictionPolicyConfig
        */
       void setEvictionPolicy(EvictionPolicyConfig evictionPolicyConfig);
  
  
  
  1.235     +1 -5      JBossCache/src/org/jboss/cache/TreeCache.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: TreeCache.java
  ===================================================================
  RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/TreeCache.java,v
  retrieving revision 1.234
  retrieving revision 1.235
  diff -u -b -r1.234 -r1.235
  --- TreeCache.java	30 Aug 2006 17:08:19 -0000	1.234
  +++ TreeCache.java	30 Aug 2006 19:09:56 -0000	1.235
  @@ -94,7 +94,7 @@
    * @author <a href="mailto:manik at jboss.org">Manik Surtani (manik at jboss.org)</a>
    * @author Brian Stansberry
    * @author Daniel Huang (dhuang at jboss.org)
  - * @version $Id: TreeCache.java,v 1.234 2006/08/30 17:08:19 msurtani Exp $
  + * @version $Id: TreeCache.java,v 1.235 2006/08/30 19:09:56 msurtani Exp $
    *          <p/>
    * @see <a href="http://labs.jboss.com/portal/jbosscache/docs">JBossCache doc</a>
    */
  @@ -3857,7 +3857,6 @@
   
         public byte[] getState(String state_id)
         {
  -         // TODO Auto-generated method stub
            return null;
         }
   
  @@ -3879,7 +3878,6 @@
   
         public void getState(String state_id, OutputStream ostream)
         {
  -         // TODO Auto-generated method stub
   
         }
   
  @@ -3924,13 +3922,11 @@
   
         public void setState(String state_id, byte[] state)
         {
  -         // TODO Auto-generated method stub
   
         }
   
         public void setState(String state_id, InputStream istream)
         {
  -         // TODO Auto-generated method stub
   
         }