Author: jboynes Date: Sat Apr 9 13:20:28 2005 New Revision: 160719 URL: http://svn.apache.org/viewcvs?view=rev&rev=160719 Log: spring clean common to remove stuff only used by twiddle
Removed: geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/CloneableObject.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/CoercionException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/Counter.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/DataConversionException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/InvalidArgumentException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/LongCounter.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/NotCoercibleException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/NullArgumentException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/ParameterizedException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/ParameterizedRuntimeException.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/Primitives.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/ThrowableHandler.java geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/ThrowableListener.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/CloneableObjectTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/CounterTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/LongCounterTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/MockThrowableListener.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/NotCoercibleExceptionTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/NullArgumentExceptionTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/ParameterizedExceptionTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/ParameterizedRuntimeExceptionTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/PrimitivesTest.java geronimo/trunk/modules/common/src/test/org/apache/geronimo/common/ThrowableHandlerTest.java Modified: geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/propertyeditor/PropertyEditors.java geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/RealmPrincipal.java Modified: geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/propertyeditor/PropertyEditors.java URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/propertyeditor/PropertyEditors.java?view=diff&r1=160718&r2=160719 ============================================================================== --- geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/propertyeditor/PropertyEditors.java (original) +++ geronimo/trunk/modules/common/src/java/org/apache/geronimo/common/propertyeditor/PropertyEditors.java Sat Apr 9 13:20:28 2005 @@ -22,7 +22,6 @@ import java.util.ArrayList; import java.util.List; -import org.apache.geronimo.common.NullArgumentException; import org.apache.geronimo.kernel.ClassLoading; /** @@ -54,7 +53,7 @@ public static PropertyEditor findEditor(final Class type) { if (type == null) { - throw new NullArgumentException("type"); + throw new IllegalArgumentException("type is null"); } PropertyEditor editor = PropertyEditorManager.findEditor(type); @@ -81,7 +80,7 @@ throws ClassNotFoundException { if (typeName == null) { - throw new NullArgumentException("typeName"); + throw new IllegalArgumentException("typeName is null"); } Class type = null; @@ -132,10 +131,10 @@ public static void registerEditor(final Class type, final Class editorType) { if (type == null) { - throw new NullArgumentException("type"); + throw new IllegalArgumentException("type is null"); } if (editorType == null) { - throw new NullArgumentException("editorType"); + throw new IllegalArgumentException("editorType is null"); } PropertyEditorManager.registerEditor(type, editorType); @@ -152,10 +151,10 @@ throws ClassNotFoundException { if (typeName == null) { - throw new NullArgumentException("typeName"); + throw new IllegalArgumentException("typeName is null"); } if (editorTypeName == null) { - throw new NullArgumentException("editorTypeName"); + throw new IllegalArgumentException("editorTypeName is null"); } ClassLoader classLoader = Thread.currentThread().getContextClassLoader(); @@ -190,7 +189,7 @@ public static void setEditorSearchPath(final List path) { if (path == null) { - throw new NullArgumentException("path"); + throw new IllegalArgumentException("path is null"); } String[] elements = (String[])path.toArray(new String[path.size()]); @@ -205,7 +204,7 @@ public static void appendEditorSearchPath(final List names) { if (names == null) { - throw new NullArgumentException("names"); + throw new IllegalArgumentException("names is null"); } if (names.size() == 0) return; @@ -223,7 +222,7 @@ public static void appendEditorSearchPath(final String[] names) { if (names == null) { - throw new NullArgumentException("names"); + throw new IllegalArgumentException("names is null"); } if (names.length == 0) return; @@ -243,7 +242,7 @@ public static void appendEditorSearchPath(final String name) { if (name == null) { - throw new NullArgumentException("name"); + throw new IllegalArgumentException("name is null"); } appendEditorSearchPath(new String[] { name }); Modified: geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/RealmPrincipal.java URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/RealmPrincipal.java?view=diff&r1=160718&r2=160719 ============================================================================== --- geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/RealmPrincipal.java (original) +++ geronimo/trunk/modules/security/src/java/org/apache/geronimo/security/RealmPrincipal.java Sat Apr 9 13:20:28 2005 @@ -20,9 +20,6 @@ import java.io.Serializable; import java.security.Principal; -import org.apache.geronimo.common.NullArgumentException; - - /** * Represents a principal in an realm. * @@ -35,8 +32,8 @@ private transient long id; public RealmPrincipal(String loginDomain, Principal principal) { - if (loginDomain == null) throw new NullArgumentException("loginDomain"); - if (principal == null) throw new NullArgumentException("principal"); + if (loginDomain == null) throw new IllegalArgumentException("loginDomain is null"); + if (principal == null) throw new IllegalArgumentException("principal is null"); this.loginDomain = loginDomain; this.principal = principal;