You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2007/05/03 09:47:49 UTC

svn commit: r534743 - /geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/Jsr77Naming.java

Author: djencks
Date: Thu May  3 00:47:48 2007
New Revision: 534743

URL: http://svn.apache.org/viewvc?view=rev&rev=534743
Log:
provide more specific fault info

Modified:
    geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/Jsr77Naming.java

Modified: geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/Jsr77Naming.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/Jsr77Naming.java?view=diff&rev=534743&r1=534742&r2=534743
==============================================================================
--- geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/Jsr77Naming.java (original)
+++ geronimo/server/trunk/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/Jsr77Naming.java Thu May  3 00:47:48 2007
@@ -66,6 +66,12 @@
     }
 
     public AbstractName createChildName(AbstractName parentAbstractName, Artifact artifact, String name, String type) {
+        if (name == null) {
+            throw new NullPointerException("No name supplied");
+        }
+        if (type == null) {
+            throw new NullPointerException("No type supplied");
+        }
         Map nameMap = new HashMap(parentAbstractName.getName());
 
         String parentType = (String) nameMap.remove(J2EE_TYPE);