Re: cvs commit:jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/runtimePageContextImpl.java

2002-11-06 Thread Ryan Lubke
I'll go ahead and give watchdog a run with this change to verify if you haven't already. On Wed, 2002-11-06 at 19:49, [EMAIL PROTECTED] wrote: costin 2002/11/06 16:49:53 Modified:jasper2/src/share/org/apache/jasper/runtime PageContextImpl.java Log:

Re: cvs commit:jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/runtimePageContextImpl.java

2002-11-06 Thread Ryan Lubke
Hi Costin, I'm still seeing failures with Watchdog's positiveErrorPageTest test. Is this working for you in your env. using a simple test case to make sure the error page works? -rl On Wed, 2002-11-06 at 19:49, [EMAIL PROTECTED] wrote: costin 2002/11/06 16:49:53 Modified:

Re: cvs commit:jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/runtimePageContextImpl.java

2002-09-30 Thread Ryan Lubke
Jan, Think it would make more sense if the message was soemthing like Name or attribute value was null? On Mon, 2002-09-30 at 19:58, [EMAIL PROTECTED] wrote: luehe 2002/09/30 16:58:58 Modified:jasper2/src/share/org/apache/jasper/runtime

Re: cvs commit:jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/runtimePageContextImpl.java

2002-06-11 Thread Kin-Man Chung
Remy, I like this much better! No private API used, and cleaner generated codes! Some nitpicks! :-) See below. There seems to be no need for a JspxState object anymore. Jspx.State.tagCount is really not used, and JspxState.out can be make local. I should just comment that out for now.