Some other miscellaneous fixes:

* Try to use the correct/detailed scopes for each syntax element
* Add some of the core built-in types (e.g. List, String, ...)
* Add "undefined" and "the empty <x>" as language constants

Author: peter-b

grammars/lcb.cson

@@ -24,7 +24,7 @@
   }
   {
     'match': '\\s*(next|end)\\s*repeat\\b'
-    'name': 'keyword.control.lcb'
+    'name': 'keyword.control.repeat.lcb'
   }
   {
     'include': '#end-block'
@@ -54,10 +54,10 @@
     'include': '#brackets'
   }
   {
-    'match': '\\b(true|false|nothing|any)\\b'
+    'match': '\\b(true|false|nothing|undefined|any|the\\s*empty\\s*(array|list|string|data))\\b'
     'captures':
       '1':
-        'name': 'constant.language.lcb'
+        'name': 'constant.language.core.lcb'
   }
   {
     'match': '\\b([tpskmxr][A-Z]\\w*)\\b'
@@ -69,7 +69,13 @@
     'match': '\\b(returns)\\b'
     'captures':
       '1':
-        'name': 'keyword.control.lcb'
+        'name': 'storage.modifier.returns.lcb'
+  }
+  {
+    'match': '\\b(Number|Integer|Real|Boolean|String|List|Array)\\b'
+    'captures':
+      '1':
+        'name': 'support.type.core.lcb'
   }
 ]
 
@@ -106,7 +112,7 @@
     'begin': '^\\s*(else\\s*if|if|else)\\b'
     'captures':
       '1':
-        'name': 'keyword.control.lcb'
+        'name': 'keyword.control.if.lcb'
     'end': '\\b(then)\\b'
     'patterns': [
       {
@@ -116,32 +122,32 @@
 
   'end-block':
     'match': '^\\s*(end(?:\\s*(?:module|library|widget|handler|if|repeat))?)'
-    'name': 'keyword.control.lcb'
+    'name': 'keyword.control.end.lcb'
 
   'repeat-block':
     'match': '^\\s*(repeat)\\b'
-    'name': 'keyword.control.lcb'
+    'name': 'keyword.control.repeat.lcb'
 
   'define-module':
     'match': '^\\s*(module|library|widget|use)(\\s+(?:\\w|[._])*)?\\b'
     'captures':
       '1':
         'name':
-          'keyword.control.lcb'
+          'keyword.other.module.lcb'
       '2':
         'name':
-          'entity.name.section.module.lcb'
+          'entity.name.module.lcb'
 
   'define-constant':
     'begin': '^\\s*constant\\b'
     'end': '\\bis\\b'
     'captures':
       '0':
-        'name': 'keyword.control.lcb'
+        'name': 'storage.type.constant.lcb'
     'patterns': [
       {
         'match': '\\b\\w+\\b'
-        'name': 'variable.constant.lcb'
+        'name': 'variable.other.constant.lcb'
       }
     ]
 
@@ -150,7 +156,7 @@
     'end': '\\bis\\b'
     'captures':
       '0':
-        'name': 'keyword.control.lcb'
+        'name': 'storage.type.metadata.lcb'
     'patterns': [
       {
         'match': '\\b\\w+\\b'
@@ -162,30 +168,32 @@
     'match': '^\\s*(property)(\\s*\\w*)?'
     'captures':
       '1':
-        'name': 'keyword.control.lcb'
+        'name': 'storage.type.property.lcb'
       '2':
         'name': 'entity.name.property.lcb'
 
   'define-variable':
     'match': '^\\s*((?:public|private)\\s+)?(variable)(?:\\s+(\\w*)(?:\\s+(as(?:\\soptional)?)(?:\\s+(\\w*))?)?)?'
     'captures':
       '1':
-        'name': 'storage.modifier.lcb'
+        'name': 'storage.modifier.access.lcb'
       '2':
-        'name': 'storage.type.lcb'
+        'name': 'storage.type.variable.lcb'
       '3':
         'name': 'variable.lcb'
       '4':
-        'name': 'keyword.control.lcb'
+        'name': 'keyword.other.as.lcb'
       '5':
         'name': 'storage.type.lcb'
 
   'define-handler':
-    'match': '^\\s*((?:(?:public|private)\\s+)?handler)(?:\\s+(\\w+))\\b'
+    'match': '^\\s*((?:public|private)\\s+)?(handler)(?:\\s+(\\w+))\\b'
     'captures':
       '1':
-        'name': 'keyword.control.lcb'
+        'name': 'storage.modifier.access.lcb'
       '2':
+        'name': 'storage.type.function.lcb'
+      '3':
         'name': 'entity.name.function.lcb'
 
   'handler-return':