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

[Remy Maucherat]

Hans Bergsten wrote:

Remy Maucherat wrote:
See my previous mail; you do _not_ need to have a package statement
for the generated servlet class. Adding it just complicates life.

Ooops, sorry, I'm trying to do plenty of things at once, and misunderstood your reply.
I'll remove the org.apache.jsp since it's not needed.

As long as there's no prefix added to the package name, yes. If you
add a package name prefix, the JspC in TC 4.1.18 (with the patch we're
talking about here) generates "package org.apache.jsp.foo;" for a
JSP page in the context-relative dir "foo" and places the class file
in "<JspC-outdir>/foo". Hence, a structure mismatch that will cause
confusion.

Indeed.

Remy


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