From: Dmitry Gutov Date: Wed, 4 May 2016 11:46:04 +0000 (+0300) Subject: Merge pull request #332 from bolivier/master X-Git-Url: https://code.delx.au/gnu-emacs-elpa/commitdiff_plain/173d1c84078afa9d0ee72d2b641354860793905f?hp=910520ee2e6f58e8ddfc3e7362492524e0565a6f Merge pull request #332 from bolivier/master Allow space before global declaration --- diff --git a/js2-mode.el b/js2-mode.el index 9515e2703..238440c8f 100644 --- a/js2-mode.el +++ b/js2-mode.el @@ -7264,7 +7264,7 @@ are ignored." when (and (eq 'block (js2-comment-node-format node)) (save-excursion (goto-char (js2-node-abs-pos node)) - (looking-at "/\\*global "))) + (looking-at "/\\* *global "))) append (js2-get-jslint-globals-in (match-end 0) (js2-node-abs-end node)))) diff --git a/tests/externs.el b/tests/externs.el index 40957f600..09a60a9d3 100644 --- a/tests/externs.el +++ b/tests/externs.el @@ -42,6 +42,13 @@ (should (equal (js2-get-jslint-globals) '("quux" "tee" "$"))))) +(ert-deftest js2-finds-jslint-globals-with-space () + (with-temp-buffer + (insert "/* global foo, bar:false, baz:true") + (js2-mode) + (should (equal (js2-get-jslint-globals) + '("foo" "bar" "baz"))))) + ;;;TODO ;; ensure that any symbols bound with the import syntax are added to the extern list ;; ensure that any symbols bound with the export syntax exist in the file scope