[jbosscache-commits] JBoss Cache SVN: r7902 - in core/trunk/src/main/java/org/jboss/cache: config and 1 other directory.

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2009-03-11 13:37:39 -0400 (Wed, 11 Mar 2009)
New Revision: 7902

Modified:
   core/trunk/src/main/java/org/jboss/cache/RegionManagerImpl.java
   core/trunk/src/main/java/org/jboss/cache/config/EvictionRegionConfig.java
Log:
JBCACHE-1489 -  Javadocs not clear on how to create basic local mode caches

Modified: core/trunk/src/main/java/org/jboss/cache/RegionManagerImpl.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/RegionManagerImpl.java	2009-03-11 17:29:26 UTC (rev 7901)
+++ core/trunk/src/main/java/org/jboss/cache/RegionManagerImpl.java	2009-03-11 17:37:39 UTC (rev 7902)
@@ -695,6 +695,7 @@
       for (EvictionRegionConfig erc : ercs)
       {
          Fqn fqn = erc.getRegionFqn();
+         if (fqn == null) throw new ConfigurationException("Regions cannot be configured with a null region fqn.  If you configured this region programmatically, ensure that you set the region fqn in EvictionRegionConfig");
          if (trace) log.trace("Creating eviction region " + fqn);
 
          if (fqn.equals(DEFAULT_REGION) || fqn.isRoot())

Modified: core/trunk/src/main/java/org/jboss/cache/config/EvictionRegionConfig.java
===================================================================
--- core/trunk/src/main/java/org/jboss/cache/config/EvictionRegionConfig.java	2009-03-11 17:29:26 UTC (rev 7901)
+++ core/trunk/src/main/java/org/jboss/cache/config/EvictionRegionConfig.java	2009-03-11 17:37:39 UTC (rev 7902)
@@ -29,6 +29,9 @@
 
 import java.lang.reflect.Method;
 
+/**
+ * It is imperative that a region Fqn is set, either via one of the constructors or using {@link #setRegionFqn(org.jboss.cache.Fqn)}.
+ */
 public class EvictionRegionConfig extends ConfigurationComponent
 {
    /**