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.com against 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 -~----------~----~----~----~------~----~------~--~---
