> Date: Thu, 18 Jul 2002 14:15:16 -0400
> From: Tim Funk <[EMAIL PROTECTED]>
> Subject: Re: cvs commit:
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper
JspCompilationContext.java
> To: Tomcat Developers List <[EMAIL PROTECTED]>
> Content-transfer-encoding: 7bit
>
> Yeah! $ is replaced by _ when JSP's are converted to java files.
>
> Question:
> When someone upgrades to this release, would it be advisable for the
> user to eliminate the work directory (or clean it) since an older
> compiled JSP now has the wrong name?
>
Yes. It'd be advisable to do so for any new release, anyways.
> Or warn them that every JSP will be recompiled?
>
By all means. Or do you think we should give the warning in the release
note?
>
> [EMAIL PROTECTED] wrote:
> > kinman 2002/07/18 10:57:27
> >
> > Modified: jasper2/src/share/org/apache/jasper Tag: tomcat_4_branch
> > JspCompilationContext.java
> > Log:
> > - use _ instead of $ to generate file and class names for jsp servlets.
> >
> > Revision Changes Path
> > No revision
> >
> >
> > No revision
> >
> >
> > 1.6.2.1 +6 -6
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/JspCompilationContext.
java
> >
> > Index: JspCompilationContext.java
> > ===================================================================
> > RCS file:
/home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/JspCompilati
onContext.java,v
> > retrieving revision 1.6
> > retrieving revision 1.6.2.1
> > diff -u -r1.6 -r1.6.2.1
> > --- JspCompilationContext.java 20 Jun 2002 22:33:57 -0000 1.6
> > +++ JspCompilationContext.java 18 Jul 2002 17:57:27 -0000 1.6.2.1
> > @@ -266,15 +266,15 @@
> > new StringBuffer(jspUri.length() - iSep);
> > if (!Character.isJavaIdentifierStart(jspUri.charAt(iSep))) {
> > // If the first char is not a legal Java letter or digit,
> > - // prepend a '$'.
> > - modifiedClassName.append('$');
> > + // prepend a '_'.
> > + modifiedClassName.append('_');
> > }
> > for (int i = iSep; i < iEnd; i++) {
> > char ch = jspUri.charAt(i);
> > if (Character.isLetterOrDigit(ch)) {
> > modifiedClassName.append(ch);
> > } else if (ch == '.') {
> > - modifiedClassName.append('$');
> > + modifiedClassName.append('_');
> > } else {
> > modifiedClassName.append(mangleChar(ch));
> > }
> >
> >
> >
> >
> > --
> > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
> >
> >
> >
>
>
>
> --
> To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
