Author: norman
Date: Mon Nov 21 20:47:08 2011
New Revision: 1204692
URL: http://svn.apache.org/viewvc?rev=1204692&view=rev
Log:
Add some more detail to the IllegalArgumentException
Modified:
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureStreamResponseImpl.java
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/StartTlsFutureResponse.java
Modified:
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureStreamResponseImpl.java
URL:
http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureStreamResponseImpl.java?rev=1204692&r1=1204691&r2=1204692&view=diff
==============================================================================
---
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureStreamResponseImpl.java
(original)
+++
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/FutureStreamResponseImpl.java
Mon Nov 21 20:47:08 2011
@@ -24,11 +24,11 @@ import java.io.InputStream;
public class FutureStreamResponseImpl extends FutureResponseImpl implements
StreamResponse{
@Override
- public synchronized void setResponse(Response response) {
+ public void setResponse(Response response) {
if (response instanceof StreamResponse) {
super.setResponse(response);
} else {
- throw new IllegalArgumentException();
+ throw new IllegalArgumentException("Response MUST be of type " +
StreamResponse.class.getName());
}
}
Modified:
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/StartTlsFutureResponse.java
URL:
http://svn.apache.org/viewvc/james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/StartTlsFutureResponse.java?rev=1204692&r1=1204691&r2=1204692&view=diff
==============================================================================
---
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/StartTlsFutureResponse.java
(original)
+++
james/protocols/trunk/api/src/main/java/org/apache/james/protocols/api/StartTlsFutureResponse.java
Mon Nov 21 20:47:08 2011
@@ -23,11 +23,11 @@ public class StartTlsFutureResponse exte
@Override
- public synchronized void setResponse(Response response) {
+ public void setResponse(Response response) {
if (response instanceof StartTlsResponse) {
super.setResponse(response);
} else {
- throw new IllegalArgumentException();
+ throw new IllegalArgumentException("Response MUST be of type " +
StartTlsResponse.class.getName());
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
