simplifications.

[xmlrouter] / impl / src / main / java / org / wamblee / xmlrouter / impl / ConfigImpl.java

diff --git a/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java b/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java
index 0324e78f8353b1dd58598e49b66ecca786b6552d..980bd07601277b1c52b1580df5e44a8a2a7f135c 100644 (file)
--- a/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java
+++ b/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java
@@ -26,9 +26,6 @@ import org.wamblee.xmlrouter.common.Id;
 import org.wamblee.xmlrouter.config.Config;
 import org.wamblee.xmlrouter.config.Identifiable;
 
-// TODO think real hard about the prefixing. We want a consistent view for clients. 
-// perhaps only provide a method to add items and hide all access to the ids. 
-
 /**
  * Default implementation of the {@link Config} interface.
  * 
@@ -36,8 +33,6 @@ import org.wamblee.xmlrouter.config.Identifiable;
  * 
  * @param <T>
  */
-// TODO make sure that each item inside this config is prefixed with the id of
-// the config.
 public abstract class ConfigImpl<T extends Identifiable<T>> implements
     ExtendedConfig<T> {
 
@@ -48,7 +43,7 @@ public abstract class ConfigImpl<T extends Identifiable<T>> implements
      * Constructs the object.
      */
     public ConfigImpl(Id<Config> aId) {
-        // TODO test for null.
+        notNull("id", aId);
         id = aId;
         registered = new ArrayList<T>();
     }