javascriptlint-commit

[javascriptlint-commit] SF.net SVN: javascriptlint: [170] trunk/tests/warnings

[<mat...@us...>]

Revision: 170
          http://javascriptlint.svn.sourceforge.net/javascriptlint/?rev=170&view=rev
Author:   matthiasmiller
Date:     2008-03-20 13:53:24 -0700 (Thu, 20 Mar 2008)

Log Message:
-----------
tests: remove syntactically invalid labels

Modified Paths:
--------------
    trunk/tests/warnings/inc_dec_within_stmt-ignore.js
    trunk/tests/warnings/inc_dec_within_stmt.js

Modified: trunk/tests/warnings/inc_dec_within_stmt-ignore.js
===================================================================
--- trunk/tests/warnings/inc_dec_within_stmt-ignore.js	2008-03-20 20:52:15 UTC (rev 169)
+++ trunk/tests/warnings/inc_dec_within_stmt-ignore.js	2008-03-20 20:53:24 UTC (rev 170)
@@ -14,7 +14,6 @@
         /*jsl:end*/
     } while (x > 0);
 
-    Label--: /*warning:bad_label*/
     do {
        x++;
     } while (x < 0);

Modified: trunk/tests/warnings/inc_dec_within_stmt.js
===================================================================
--- trunk/tests/warnings/inc_dec_within_stmt.js	2008-03-20 20:52:15 UTC (rev 169)
+++ trunk/tests/warnings/inc_dec_within_stmt.js	2008-03-20 20:53:24 UTC (rev 170)
@@ -55,7 +55,6 @@
             /*jsl:end*/
         } while (x > 0);
 
-        Label--: /*warning:bad_label*/
         do {
            x++;
         } while (x < 0);


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

[javascriptlint-commit] SF.net SVN: javascriptlint: [174] trunk/tests/warnings

[<mat...@us...>]

Revision: 174
          http://javascriptlint.svn.sourceforge.net/javascriptlint/?rev=174&view=rev
Author:   matthiasmiller
Date:     2008-03-20 14:30:17 -0700 (Thu, 20 Mar 2008)

Log Message:
-----------
tests: missing_default_case is now reported at the top of the switch, and missing_break_for_last_case is now reported at the top of the case statement

Modified Paths:
--------------
    trunk/tests/warnings/missing_break_for_last_case.js
    trunk/tests/warnings/missing_default_case.js

Modified: trunk/tests/warnings/missing_break_for_last_case.js
===================================================================
--- trunk/tests/warnings/missing_break_for_last_case.js	2008-03-20 21:28:43 UTC (rev 173)
+++ trunk/tests/warnings/missing_break_for_last_case.js	2008-03-20 21:30:17 UTC (rev 174)
@@ -1,15 +1,15 @@
 /*jsl:option explicit*/
 function missing_break_for_last_case(i) {
     switch (i) {
-      default:
+      default: /*warning:missing_break_for_last_case*/
         /*missing break at end of switch (without code)*/
-    } /*warning:missing_break_for_last_case*/
+    }
 
     /*missing break at end of switch (with code)*/
     switch (i) {
-      default:
+      default: /*warning:missing_break_for_last_case*/
          i++;
-    } /*warning:missing_break_for_last_case*/
+    }
 
     /*ok because of fallthru*/
     switch (i) {

Modified: trunk/tests/warnings/missing_default_case.js
===================================================================
--- trunk/tests/warnings/missing_default_case.js	2008-03-20 21:28:43 UTC (rev 173)
+++ trunk/tests/warnings/missing_default_case.js	2008-03-20 21:30:17 UTC (rev 174)
@@ -3,10 +3,10 @@
     var i, s;
 
     /*missing default case*/
-    switch (i) {
+    switch (i) { /*warning:missing_default_case*/
       case 1:
         return 1;
-    } /*warning:missing_default_case*/
+    }
 
     /* ambivalence - allow fallthru but don't enforce it */
     switch (i) {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.