https://issues.apache.org/bugzilla/show_bug.cgi?id=17441
[email protected] changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #11 from [email protected] 2009-01-15 11:43:20 PST --- I know that 'stripjavacomments' is intended for Java, but I tried it out on JavaScript. There is still an issue with double-quoted // in that environment, and with /* in a single quoted string. I'm not surprised that a single quoted string is a problem, since Java doesn't use single quotes for strings. That's easily fixed by using double quotes. But I am surprised by the continuing problem with double slash inside a double quoted string. Yes, I did download 1.7.1 just yesterday and tried it out, the issue is still there. Both examples are drawn from "prototype.js"; a version of which came with Scriptaculous. The particular line I'm talking about is around 3974, and reads: document.write("<script id=__onDOMContentLoaded defer src=//:><\/script>"); If the problems arising from using this filter on JavaScript can be easily solved, that would be a great benefit to all. I haven't reopened the bug, because I'm not sure if anyone else feels that not handling JavaScript is really a problem for a filter designed for Java. <grin> -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
