Tobie, I see that other tools complains with some of the syntax that has been kept for years, not only JSLint.
I wonder why Caja is taken in so much more consideration than a practice that always existed in the programming world. Anyway nice these minor annoyances were fixed...So thanks Tobie and Caja ;-) Diego On 13 Mar, 03:00, Tobie Langel <tobie.lan...@gmail.com> wrote: > 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 -~----------~----~----~----~------~----~------~--~---
