|
From: <jue...@we...> - 2004-08-20 20:21:10
|
Just changed it - thanks for the suggestion! This also covers reflection =
errors now. (Errors thrown by the target method were already covered by =
the InvocationTargetException before.)
=20
Juergen
=20
________________________________
Von: spr...@li... im Auftrag =
von Martin Kersten
Gesendet: Fr 20.08.2004 21:57
An: spr...@li...
Betreff: [Springframework-developer] -> HttpInvoker / spring.remote =
suggestion
Hi folks,
Checking the Spring 1.1 RC2 remote code, I have noticed,
that the RemoteInvocationResult class is referring to Exception
rather than Throwable.
Visiting the InvocationTargetException the code reads:
/**
* Get the thrown target exception.
*
* <p>This method predates the general-purpose exception chaining
facility.
* The {@link Throwable#getCause()} method is now the preferred =
means of
* obtaining this information.
*
* @return the thrown target exception (cause of this exception).
*/
public Throwable getTargetException() {
return target;
}
So I guess it would be more consistent to use Throwable instead
of Exception since the Throwable would be more abstract (thus
more flexibile).
Summary:
public class RemoteInvocationResult implements Serializable {
private Object value;
private Exception exception;
should be changed into:
public class RemoteInvocationResult implements Serializable {
private Object value;
private Throwable exception;
Cheers,
Martin (Kersten)
-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
_______________________________________________
Springframework-developer mailing list
Spr...@li...
https://lists.sourceforge.net/lists/listinfo/springframework-developer
|