You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jb...@apache.org on 2005/04/09 22:20:30 UTC
svn commit: r160719 - in geronimo/trunk/modules:
common/src/java/org/apache/geronimo/common/
common/src/java/org/apache/geronimo/common/propertyeditor/
common/src/test/org/apache/geronimo/common/
security/src/java/org/apache/geronimo/security/
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;