2011/6/28 Mark Thomas <ma...@apache.org>: > >> IMHO it is '_' that was treated as special char, because mangleChar() >> prepends '_' as well. Otherwise the mangling will be unreversible. > > Does that matter? Do we need to able to reverse this?
To avoid collisions between mangled names and names that did not need to be mangled. >> The use case of '.' is just to use shorter name for a frequent use case. >> E.g. "index.jsp" -> "index_jsp.class". > > It would be nice to keep that. Yes. It would be some surprise if names of (pre)compiled jsps will change between minor versions. > I suppose we could add a > periodToUnderscore parameter to makeJavaIdentifier() and essnetially > have both approaches. The current one for file names, the new one for > variable names. Or better just add a separate method. Best regards, Konstantin Kolinko --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org