[JavaScript] Fix scoping of called private method names. (sublimehq#2487

)
mitranim · Mar 20, 2022 · dd83d1b · dd83d1b
1 parent cb18379
commit dd83d1b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/JavaScript/JavaScript.sublime-syntax b/JavaScript/JavaScript.sublime-syntax
@@ -1909,7 +1909,7 @@ contexts:
         - function-call-arguments
         - literal-variable
 
-    - match: (?={{identifier}}\s*(?:{{dot_accessor}}\s*{{identifier}}\s*)+(?:{{dot_accessor}})?\()
+    - match: (?={{identifier}}\s*(?:{{dot_accessor}}\s*#?{{identifier}}\s*)+(?:{{dot_accessor}})?\()
       set:
         - call-method-meta
         - function-call-arguments
@@ -1946,6 +1946,11 @@ contexts:
     - match: '{{identifier}}'
       scope: variable.function.js
       pop: true
+    - match: '(#){{identifier}}'
+      scope: variable.function.js
+      captures:
+        1: punctuation.definition.js
+      pop: true
     - include: else-pop
 
   literal-variable:
@@ -2372,7 +2377,7 @@ contexts:
   object-property:
     - match: |-
         (?x)(?=
-          {{identifier}}
+          \#?{{identifier}}
           \s* = \s*
           {{either_func_lookahead}}
         )
@@ -2383,7 +2388,7 @@ contexts:
 
     - include: support-property
 
-    - match: '(?={{identifier}}\s*(?:{{dot_accessor}})?\()'
+    - match: '(?=#?{{identifier}}\s*(?:{{dot_accessor}})?\()'
       set: call-method-name
 
     - include: object-property-base

diff --git a/JavaScript/tests/syntax_test_js.js b/JavaScript/tests/syntax_test_js.js
@@ -1377,6 +1377,12 @@ sources.DOM
     .status()
     // ^ meta.function-call.method variable.function
 
+    foo.#bar();
+//  ^^^^^^^^^^ meta.function-call.method.js
+//      ^^^^ variable.function.js
+//      ^ punctuation.definition.js
+//          ^^ meta.group.js
+
 return new Promise(resolve => preferenceObject.set({value}, resolve));
 //                                                                  ^ meta.function-call.constructor punctuation.section.group.end