ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject DO NOT REPLY [Bug 17441] stripjavacomments strips too much
Date Thu, 15 Jan 2009 19:43:21 GMT changed:

           What    |Removed                     |Added
                 CC|                            |

--- Comment #11 from  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

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

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:
------- You are receiving this mail because: -------
You are on the CC list for the bug.

View raw message