Add a test for #335 as well

Merge branch 'dgreensp-object-rest-spread'

Backport for fix for #335

Merge pull request #332 from bolivier/master

Allow space before global declaration

Merge pull request #333 from daniellandau/master

Change warning message to not conflict with spec

Change warning message to not conflict with spec

The ECMAScript spec says
(http://www.ecma-international.org/ecma-262/5.1/#sec-10.5) that
declaring an already declared variable is silently ignored, so claiming
it's a TypeError is not correct. It should still be highlighted as a
warning because it's not something that any program should do and with
let redeclaration actually is a TypeError.

Allow space before global declaration

Declare vars created with renaming destructuring

Fixes regression from 0555a8a3.

Add :warnings-count keyword argument

Merge pull request #328 from XeCycle/iname-color

Colorize imported names as variables

Colorize imported names as variables

Merge pull request #327 from XeCycle/ckw-as

Colorize contextual keyword as in import & export

Colorize contextual keywords as & from in import & export

Merge pull request #326 from phst/compiler-warning-25

Fix a byte compilation warning on Emacs 25

Fix a byte compilation warning on Emacs 25

Fix more remaining failures

Closes #324

Merge pull request #325 from phst/all-tests

Run all tests on 'make test'

Run all tests on 'make test'

Fix the straggler

Fix the tests broken in the previous commit

Use idle timer to reparse, even when the buffer's just been opened

Fixes #20, in yet another way.

Also speeds up Emacs 25's xref-find-references and project-find-regexp.

Special-case unary + and -

Fixes #322

Merge pull request #289 from dgreensp/fix-array-rest

Fix array destructuring including triple-dot

js2-indent-operator-re, js2-declaration-keyword-re: Use symbols boundaries

Fixes #319

Fix computed property generator methods indentation

Closes #317

Merge pull request #308 from zmwangx/jsdoc-callback-func-method

Add support for JSDoc @callback, @func and @method tags

Add support for JSDoc @callback, @func and @method tags

See http://usejsdoc.org/tags-callback.html for documentation of the
@callback tag; see http://usejsdoc.org/tags-function.html for
documentation of the @func and @method tags (both are synonyms of the
@function tag).

Closes #307.

Merge pull request #304 from jacksonrayhamilton/fix-void-async-await

Fix void async await

Simplify await parsing

Fix compiler warning

Refactor async arrow function matching

Refactor await parsing to only parse once

Fix typo

Add tests for of parsing

Fix await parsing

Fix async parsing

Fix void parsing

Remove js2-make-await as unused

Record class declarations for IMenu

Closes #303

And move its definition below

js2-class-node: Inherit from js2-object-node

js2-mode-extend-comment: Don't check that the next line is also commented

Fixes #302

Merge pull request #297 from Christoph-D/master

Recognize for-of and for-in loops with const

Recognize for-of and for-in loops with const

Declarations in for-of and for-in loops may also be "const".  See
http://www.ecma-international.org/ecma-262/6.0/#sec-runtime-semantics-bindinginstantiation

Merge pull request #294 from lunaryorn/patch-1

Use SVG for Travis CI badge

Use SVG for Travis CI badge

Let's be Retina friendly.

Fix a byte-compilation warning

Implement object literal rest/spread (...)

As discussed in #245, this syntax is not part of ES6/2015, but it is
a stage 2 proposal, meaning the committee expects it to be included
in the standard.  It's already seeing heavy use by e.g. the React
community.

This change causes `...expr` in an object literal to be parsed as a
unary-node element of an object-node, and modifies places where an
element of an object-node is expected to be an infix-node.  Note that
only weak typing assumptions are made already about object-node elems.

In addition, this change implements proper name extraction in
object literal destructuring.  The cases are:

1. `let {a} = x;`
2. `let {a: b} = x;` (should define `b`)
3. `let {a = b} = x;` (should define `a`)
4. `let {...a} = x;` (should define `a`)

This change fixes case (2), which was broken, and adds case (4).

Merge pull request #290 from XeCycle/export-async-fn

Also parse async function statement in export node

Also parse async function statement in export node

AsyncFunctionDeclaration is added as a
HoistableDeclaration (http://tc39.github.io/ecmascript-asyncawait/#HoistableDeclaration),
which shall be covered in productions "export Declaration" and "export
default HoistableDeclaration".

Do not print trailing semicolon for export function and class

Fix array destructuring including triple-dot

This change enables proper parsing of:
* `let [a, b, ...c] = d;`
* `let [[x,y] = [1,2]] = z;`

The first case is simply the "rest" part of "array rest/spread," which
lacked the appropriate cases in `js2-parse-array-literal` and
`js2-define-destruct-symbols`.  There's also new logic to make sure that
in the destructuring (rest) case, the rest element comes last (while
spread can occur anywhere).

The removal of the `;; destructuring bind` case of
`js2-parse-array-literal` may require some explanation.  It was doing
more harm than good.  For example, it assumed that only identifiers
have defaults.  Also, calling `js2-parse-destruct-primary-expr` is
unnecessary, since `js2-is-in-destructuring` is already true.  Rather
than add the triple-dot logic in two places, it was simpler to stop
distinguishing between destructuring and non-destructuring at this
level, because the grammars are extremely similar.  In addition, wacky
"vars" as in `let [f()] = x` throw an error anyway when they hit
`js2-define-destruct-symbols`.

Adjust js2-print-export-node

#287

Merge pull request #287 from dgreensp/fix-export-decls

Fix semicolons and scoping in exported decls

Exported decls print a little funny; update tests

Fix semicolons and scoping in exported decls

In the following code, no semicolons should be considered missing,
and all of A,B,C,D should be considered declared.

```
export function A() {}
export class B {}
export default function C() {}
export default class D {}

var x = [A, B, C, D];
```

s/will/would

Merge pull request #283 from XeCycle/master-with-async

Adapt old async to master & async methods

Add NEWS entry for async/await

Marking experimental because there are some TODOs in the code, and the
proposal is not yet final anyway.

Parse async methods

Refs https://github.com/mooz/js2-mode/issues/185#issuecomment-107710557

Add setImmediate and clearImmediate to js2-node-externs

Closes #286

Merge pull request #285 from dgreensp/fix-param-destruct

Fix parsing of param with destructure and default

Fix parsing of param with destructure and default

`function foo(bar, {a = {}} = {}) {/**/}`

Merge pull request #284 from XeCycle/0o-octals

Allow 0o octals in strict mode

Allow 0o octals in strict mode

0o octals are allowed in ES6 strict mode, while octals like 0123 are
called LegacyOctalIntegerLiteral and remain disallowed.

Move method type handling to property METHOD_TYPE

Relying on js2-node-type requires a token number for each type of
method, but we did not define a js2-ASYNC for the coming async methods.

Only getter and setter are moved, generator is not.

Adapt async/await changes to latest master

Resolved conflicts:
- fix call signatures to js2-parse-function
- fix js2-AWAIT number
- adapt to rename of getter-setter-node to method-node

Merge pull request #278 from redguardtoo/master

`js2-print-json-path' added

`js2-print-json-path' added

Support .jsx file extension too in the example

Merge pull request #282 from jacksonrayhamilton/js2-jsx-mode

Add js2-jsx-mode

Add js2-jsx-mode usage instructions

Add NEWS entry for js2-jsx-mode

Copy JSX indentation logic to js2-old-indent.el

Add js2-jsx-mode

Update MELPA URL

Merge pull request #280 from XeCycle/binding-pattern-initializer

Add default parameters support to destructuring

Merge pull request #281 from robbyoconnor/patch-1

Add melpa badges

Add melpa badges

Add melpa badges to the README

Report error for binding initializer outside destructuring

This reports error for both object literals and class bodies, but only
tests object literals; because if ES7 class property initializers are
added, the error no longer makes sense and needs another message,
e.g. missing semicolon after property initializer.

Support initializer in destructuring

Also known as default arguments.

Merge pull request #279 from XeCycle/class-semicolon-elem

Allow semicolons in a class body

Allow semicolons in a class body

Merge pull request #276 from XeCycle/methods

Rename getter-setter-node to method-node

Add NEWS entry about rename to method-node

Refactor: rename getter-setter-node to method-node

Add test on shorthand method name and scoping

Refactor: unify computed property name handling

Handle open-curlies after newline

Fixes #277

js2-parse-highlight-prop-get: Take care now to overwrite the js2-function-call face

#272

Add a test for the latest change

Don't misindent generator methods

Fixes #275 in the old-indent subpackage; Emacs 25 snapshot users
should update to the latest version.

Also highlight property lookups with the same face

#272

Define js2-object-property face

#272

Fix 'make all' on older emacsen

Closes #268

Merge pull request #266 from jacksonrayhamilton/no-getter-setter-duplicate

Don't treat getter / setter pairs as duplicates

Don't treat getter / setter pairs as duplicates

Fixes #264

Rephrase

Add NEWS and bump the version

Explicitly remap js-find-symbol

#159