[PATCH] Root cause lost in Jsp and ServletException

[Costin Manolache]

this is for servetapi_4.

Same applies to servletapi_5 ( I already sent a patch few weeks ago ).

The problem is that JDK1.4 stack traces can't find the root cause.

AFAIK nothing in the servlet or JSP specs requires breaking the normal
contract for exceptions - which is to pass the root cause to the parent.

Costin

Index: src/share/javax/servlet/ServletException.java
===================================================================
RCS file:
/home/cvs/jakarta-servletapi-4/src/share/javax/servlet/ServletException.java,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 ServletException.java
--- src/share/javax/servlet/ServletException.java       9 Jan 2001 03:24:16 -0000
1.1.1.1
+++ src/share/javax/servlet/ServletException.java       4 Feb 2003 19:00:30 -0000
@@ -129,7 +129,7 @@
      */
     
     public ServletException(String message, Throwable rootCause) {
-       super(message);
+       super(message, rootCause);
        this.rootCause = rootCause;
     }
 
@@ -158,7 +158,7 @@
      */
 
     public ServletException(Throwable rootCause) {
-       super(rootCause.getLocalizedMessage());
+       super(rootCause.getLocalizedMessage(), rootCause);
        this.rootCause = rootCause;
     }
   
Index: src/share/javax/servlet/jsp/JspException.java
===================================================================
RCS file:
/home/cvs/jakarta-servletapi-4/src/share/javax/servlet/jsp/JspException.java,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 JspException.java
--- src/share/javax/servlet/jsp/JspException.java       9 Jan 2001 03:24:20 -0000
1.1.1.1
+++ src/share/javax/servlet/jsp/JspException.java       4 Feb 2003 19:00:31 -0000
@@ -106,7 +106,7 @@
      */
     
     public JspException(String message, Throwable rootCause) {
-       super(message);
+       super(message, rootCause);
        this.rootCause = rootCause;
     }
 
@@ -132,7 +132,7 @@
      */
 
     public JspException(Throwable rootCause) {
-       super(rootCause.getLocalizedMessage());
+       super(rootCause.getLocalizedMessage(), rootCause);
        this.rootCause = rootCause;
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]