jsdoc3/jsdoc — pull request #139 by hegemonic (2012-07-04 21:51:36+00:00)
Title: Make JSDoc JSHint-clean (part 1)

I'd like to contribute to JSDoc, but I'd also like to use [JSHint](http://www.jshint.com/about/) as I code to help me avoid silly errors. However, the current version of JSDoc is not JSHint-clean.

This is the first in a series of pull requests that will allow JSDoc to be used with JSHint. This pull request includes:

* A new test for JSHint errors. The test examines all of JSDoc's .js files except:
* Third-party modules in `Jake/` and `node_modules/`.
* Tests in `test/`.
* `lib/rhino-shim.js`, which (necessarily) redefines some variables that JSHint considers read-only.
* A `.jshintrc` config file that turns off every JSHint test that can be disabled.
* Fixes for all JSHint tests that cannot be disabled. I tried to make one commit per test, so that it's obvious what's being changed and why.

I think these changes will be fairly uncontroversial, with the possible exception of 77f505f, which eliminates the use of [destructuring assignment](https://developer.mozilla.org/en/New_in_javascript_1.7#Destructuring_assignment_%28Merge_into_own_page.2Fsection%29). JSHint doesn't understand this syntax. It turns out that Node.js/V8 doesn't either, so we'll have to make this change eventually anyhow.

In future pull requests, I'll turn on specific JSHint tests and provide fixes so that JSDoc passes each test. I don't plan to enable every single test, but I'll probably enable most of them.

If you have any comments, please let me know!