490219 - Enable jslint options

Status: RESOLVED FIXED

(Skywriter Graveyard :: Editor, enhancement)

Description

[Chris J]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3) Gecko/20090305 Firefox/3.1b3
Build Identifier: 

Expand jslint setting to enable options, e.g. jslint forin=true safe=true


Reproducible: Always

Comment 1

[Gordon P. Hemsley [:GPHemsley]]

This is a logical extension of bug 486949, so confirming.

Comment 2

[Chris J]

Attachment: adds jslint options support via "set jslint {}"

This patch adds jslint options support. See http://www.jslint.com/lint.html for all the available options. They are entered as a json string to the "jslint" setting, for example type this in the command line:

"set jslint {eqeqeq:true}" 

to create a warning for all uses of double equals in javascript (since triple equals is safer).

I've tried to make the code extensible by treating it as one of a number of "parseOptions" that could be run for different languages.

Comment 3

[Dion Almaer]

The patch collided with the other one, so I did it by hand and it is in tip. Please give it a once over!