JSLint and automatic indentation don't agree with non-default tabSize

[0b10011]

Steps to reproduce

1. Create a JavaScript file with it's contents set to var a = 0, b = 0;
2. Set tabSize to 2
3. Place the cursor after , and press Enter
4. Save / Run JSLint

Expected results

JSLint should have 0 problems.

Actual results

JSLint reports 1 problem:

Expected 'b' at column 5, not column 9.

[zaggino]

It already has this code - what's your spaceUnits setting?

if (!options.indent) {
            // default to using the same indentation value that the editor is using
            options.indent = PreferencesManager.get("spaceUnits");
        }

[zaggino]

I simulated this with using tabSize for indent options but this only changes the error from 5-9 to Expected 'b' at column 3, not column 5. because it is automatically formatted to

var a = 0, 
    b = 0;

but JSLint with indent=2 would like to see it like this:

var a = 0, 
  b = 0;

[0b10011]

spaceUnits is set to 4 for me (I changed tabSize via the input in the status bar).

[zaggino]

I put up a PR - it doesn't solve the indentation issue after var but it does pass correct indent option to JSLint.

[dangoor]

Could the placement of the var on the next line be a result of #7218?

[zaggino]

It's quite possible. We'll see after it gets fixed.

[redmunds]

I think this indenting is caused by CodeMirror smartIndent addon which is on by default and can be turned off with "smartIndent": false.

[zaggino]

@bfrohs can you re-test with latest master?

[0b10011]

@zaggino Fixed, thanks!