Merge pull request #280 from JuliaEditorSupport/sp/better-public

fix: only tokenize public at line start as kw

Author: pfitzseb

grammars/julia.cson

@@ -286,7 +286,7 @@ repository:
         name: 'keyword.control.export.julia'
       }
       {
-        match: '\\b(?<![:_])(?:public)\\b'
+        match: '^(?:public)\\b'
         name: 'keyword.control.public.julia'
       }
       {

grammars/julia.json

@@ -318,7 +318,7 @@
           "name": "keyword.control.export.julia"
         },
         {
-          "match": "\\b(?<![:_])(?:public)\\b",
+          "match": "^(?:public)\\b",
           "name": "keyword.control.public.julia"
         },
         {

grammars/julia.template.json

@@ -318,7 +318,7 @@
           "name": "keyword.control.export.julia"
         },
         {
-          "match": "\\b(?<![:_])(?:public)\\b",
+          "match": "^(?:public)\\b",
           "name": "keyword.control.public.julia"
         },
         {

grammars/julia_vscode.json

@@ -318,7 +318,7 @@
           "name": "keyword.control.export.julia"
         },
         {
-          "match": "\\b(?<![:_])(?:public)\\b",
+          "match": "^(?:public)\\b",
           "name": "keyword.control.public.julia"
         },
         {

test/test.js

@@ -756,6 +756,27 @@ describe('Julia grammar', function () {
             },
         ])
     })
+    it("does not tokenize public in non-toplevel scopes", function () {
+        const tokens = tokenize(grammar, "2 + public")
+        compareTokens(tokens, [
+            {
+                value: "2",
+                scopes: ["constant.numeric.julia"]
+            },
+            {
+                value: " ",
+                scopes: []
+            },
+            {
+                value: "+",
+                scopes: ["keyword.operator.arithmetic.julia"]
+            },
+            {
+                value: " public",
+                scopes: []
+            },
+        ])
+    })
     it("tokenizes symbols", function () {
         const tokens = tokenize(grammar, ":à_b9!")
         compareTokens(tokens, [