Some of those have been fixed in my branch for Caja compliance.
On Mar 12, 11:10 pm, Robert Kieffer <bro...@gmail.com> wrote:
> Would it be worth doing some sort of javascript lint'ing as part of
> the 'dist' or 'test' tasks? ... or as part of whatever the final
> release processis? When I run http://www.javascriptlint.comagainst
> 1.6.0.3 or trunk, even using a fairly conservative config, I see a
> healthy dose of warnings (included below).
>
> In addition to dozens of 'missing semicolon' warnings, I also see
> stuff like this (on latest trunk dist):
>
> prototype.js|450| lint warning: multiple statements separated by
> commas (use semicolons?)
> prototype.js|2206| lint warning: multiple statements separated by
> commas (use semicolons?)
> prototype.js|2745| lint warning: the else statement could be matched
> with one of multiple if statements (use curly braces to indicate
> intent)
> prototype.js|3269| lint warning: multiple statements separated by
> commas (use semicolons?)
> prototype.js|3577| warning: variable nodes hides argument
> prototype.js|3932| lint warning: multiple statements separated by
> commas (use semicolons?)
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Prototype: Core" group.
To post to this group, send email to prototype-core@googlegroups.com
To unsubscribe from this group, send email to
prototype-core-unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/prototype-core?hl=en
-~----------~----~----~----~------~----~------~--~---
