ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 17441] stripjavacomments strips too much
Date Thu, 15 Jan 2009 19:43:21 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=17441


Spindizzy_Wizard@yahoo.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Spindizzy_Wizard@yahoo.com




--- Comment #11 from Spindizzy_Wizard@yahoo.com  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.

Mime
View raw message