Linting errors + 'bench' not working on Windows

[joshq00]

• Added .jshintrc
• Removed unneeded variables / matched quotes / == to ===
• benchmark/parse_map.js
  • was trying to split by '\r\n' on Windows. Changed to /\r?\n/
  • instead of calling toString, read file by utf8 encoding

[joshq00]

Mine is unset. But, either way, the regular expression will work, while the EOL assumes the files have the same line endings your os uses

[imor]

What do smarttabs and trailing options do? I couldn't find them in the jshint docs.

[joshq00]

Smart tabs: http://jshint.com/docs/faq/

Trailing: looks like this was removed with v2.5, it warned for trailing
whitespace on lines

On Sunday, February 1, 2015, Raminder Singh [email protected]
wrote:

In .jshintrc
#88 (comment):

@@ -0,0 +1,17 @@
+{

• "browser": false,
• "node": true,
• "globals": {
• "require": true
• },
• "predef": {
• },
• "eqeqeq": true,
• "quotmark": true,
• "smarttabs": true,
• "trailing": true,

What do smarttabs and trailing options do? I couldn't find them in the
jshint docs.

Reply to this email directly or view it on GitHub
https://github.com/qiao/PathFinding.js/pull/88/files#r23895774.

[imor]

@joshq00 Everything looks fine to me. Thanks for the patch 👍
@qiao Any questions you have or should I go ahead and merge?

[joshq00]

?