Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Invalid alidation of Missing semicolon #69

Open
GoogleCodeExporter opened this issue Jun 17, 2015 · 4 comments
Open

Invalid alidation of Missing semicolon #69

GoogleCodeExporter opened this issue Jun 17, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Run gjslint validation on the following code
  var f = function() {
    var m = 1,
      fn = function() {

      },
      b = 12;
  };


  var f2 = function() {
    var p1 = 1, p2 = 2;
  };

What is the expected output? What do you see instead?
 - no errors
Instead there is a notification 
  Line 5, E:0011: Missing semicolon after function assigned to a variable


What version of the product are you using? On what operating system?


Please provide any additional information below.

Run fixjsstyle
    Fixed 1 errors

Code looks like:
      };,
      b = 12;
what generates real javascript error

Original issue reported on code.google.com by [email protected] on 30 May 2013 at 8:37

@GoogleCodeExporter
Copy link
Author

For clarity/readability, these should be separate var statements anyways, but 
it's a valid issue that fixjsstyle should not insert a semicolon here.

Original comment by [email protected] on 15 Jan 2015 at 8:51

@GoogleCodeExporter
Copy link
Author

Issue 83 has been merged into this issue.

Original comment by [email protected] on 15 Jan 2015 at 8:52

@GoogleCodeExporter
Copy link
Author

Issue 60 has been merged into this issue.

Original comment by [email protected] on 15 Jan 2015 at 8:53

@GoogleCodeExporter
Copy link
Author

Should fix this.

Original comment by [email protected] on 7 Apr 2015 at 5:19

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant