Recognize '&&' and '||' as illegal operators

1st1 · 1st1 · commit 1ba703f34654 · 2015-10-24T02:39:22.000-04:00
diff --git a/grammars/MagicPython.cson b/grammars/MagicPython.cson
@@ -290,6 +290,9 @@ repository:
       {
         include: "#lambda"
       }
+      {
+        include: "#illegal-operator"
+      }
       {
         include: "#operator"
       }
@@ -332,9 +335,6 @@ repository:
       {
         include: "#special-names"
       }
-      {
-        include: "#illegal-operator"
-      }
       {
         include: "#illegal-names"
       }
@@ -1677,11 +1677,15 @@ repository:
   "illegal-operator":
     patterns: [
       {
-        name: "invalid.illegal.character.python"
+        name: "invalid.illegal.operator.python"
+        match: "&&|\\|\\|"
+      }
+      {
+        name: "invalid.illegal.operator.python"
         match: "[?$]"
       }
       {
-        name: "invalid.illegal.character.python"
+        name: "invalid.illegal.operator.python"
         comment: "We don't want `!` to flash when we're typing `!=`"
         match: "!\\b"
       }
diff --git a/grammars/MagicPython.syntax.yaml b/grammars/MagicPython.syntax.yaml
@@ -233,6 +233,7 @@ repository:
       - include: '#regexp'
       - include: '#string'
       - include: '#lambda'
+      - include: '#illegal-operator'
       - include: '#operator'
       - include: '#curly-braces'
       - include: '#item-access'
@@ -247,7 +248,6 @@ repository:
       - include: '#magic-variable-names'
       - include: '#member-access'
       - include: '#special-names'
-      - include: '#illegal-operator'
       - include: '#illegal-names'
       - include: '#special-variables'
       - include: '#ellipsis'
@@ -1149,9 +1149,11 @@ repository:
 
   illegal-operator:
     patterns:
-      - name: invalid.illegal.character.python
+      - name: invalid.illegal.operator.python
+        match: '&&|\|\|'
+      - name: invalid.illegal.operator.python
         match: '[?$]'
-      - name: invalid.illegal.character.python
+      - name: invalid.illegal.operator.python
         comment: We don't want `!` to flash when we're typing `!=`
         match: '!\b'
 
diff --git a/grammars/MagicPython.tmLanguage b/grammars/MagicPython.tmLanguage
@@ -452,6 +452,10 @@
             <key>include</key>
             <string>#lambda</string>
           </dict>
+          <dict>
+            <key>include</key>
+            <string>#illegal-operator</string>
+          </dict>
           <dict>
             <key>include</key>
             <string>#operator</string>
@@ -508,10 +512,6 @@
             <key>include</key>
             <string>#special-names</string>
           </dict>
-          <dict>
-            <key>include</key>
-            <string>#illegal-operator</string>
-          </dict>
           <dict>
             <key>include</key>
             <string>#illegal-names</string>
@@ -2640,13 +2640,19 @@ indirectly through syntactic constructs
         <array>
           <dict>
             <key>name</key>
-            <string>invalid.illegal.character.python</string>
+            <string>invalid.illegal.operator.python</string>
+            <key>match</key>
+            <string>&amp;&amp;|\|\|</string>
+          </dict>
+          <dict>
+            <key>name</key>
+            <string>invalid.illegal.operator.python</string>
             <key>match</key>
             <string>[?$]</string>
           </dict>
           <dict>
             <key>name</key>
-            <string>invalid.illegal.character.python</string>
+            <string>invalid.illegal.operator.python</string>
             <key>comment</key>
             <string>We don&apos;t want `!` to flash when we&apos;re typing `!=`</string>
             <key>match</key>
diff --git a/misc/scopes b/misc/scopes
@@ -30,12 +30,12 @@ entity.other.inherited-class.python
 invalid.deprecated.backtick.python
 invalid.deprecated.prefix.python
 invalid.illegal.annotation.python
-invalid.illegal.character.python
 invalid.illegal.dec.python
 invalid.illegal.decorator.python
 invalid.illegal.line.continuation.python
 invalid.illegal.name.python
 invalid.illegal.newline.python
+invalid.illegal.operator.python
 invalid.illegal.prefix.python
 keyword.codetag.notation.python
 keyword.control.flow.python
diff --git a/test/illegals/illegal1.py b/test/illegals/illegal1.py
@@ -18,11 +18,11 @@ def           : meta.function.python, source.python, storage.type.function.pytho
 :             : meta.function.python, punctuation.section.function.begin.python, source.python
               : source.python
 pass          : keyword.control.flow.python, source.python
-$             : invalid.illegal.character.python, source.python
-?             : invalid.illegal.character.python, source.python
+$             : invalid.illegal.operator.python, source.python
+?             : invalid.illegal.operator.python, source.python
 a             : source.python
 =             : keyword.operator.assignment.python, source.python
-$             : invalid.illegal.character.python, source.python
+$             : invalid.illegal.operator.python, source.python
 (             : punctuation.parenthesis.begin.python, source.python
 '             : punctuation.definition.string.begin.python, source.python, string.quoted.single.python
 .class        : source.python, string.quoted.single.python
@@ -40,8 +40,8 @@ def           : meta.function.python, source.python, storage.type.function.pytho
 b             : source.python
 =             : keyword.operator.assignment.python, source.python
               : source.python
-!             : invalid.illegal.character.python, source.python
+!             : invalid.illegal.operator.python, source.python
 some_ruby     : source.python
-?             : invalid.illegal.character.python, source.python
+?             : invalid.illegal.operator.python, source.python
 #             : comment.line.number-sign.python, punctuation.definition.comment.python, source.python
  hey ;)       : comment.line.number-sign.python, source.python
diff --git a/test/illegals/illegal2.py b/test/illegals/illegal2.py
@@ -0,0 +1,19 @@
+a&&b||c
+a &&= a
+b ||= c
+
+
+
+a             : source.python
+&&            : invalid.illegal.operator.python, source.python
+b             : source.python
+||            : invalid.illegal.operator.python, source.python
+c             : source.python
+a             : source.python
+&&            : invalid.illegal.operator.python, source.python
+=             : keyword.operator.assignment.python, source.python
+ a            : source.python
+b             : source.python
+||            : invalid.illegal.operator.python, source.python
+=             : keyword.operator.assignment.python, source.python
+ c            : source.python
diff --git a/test/regexp/python2.py b/test/regexp/python2.py
@@ -13,7 +13,7 @@
               : invalid.illegal.newline.python, source.python, string.regexp.quoted.single.python
               : source.python
 (             : punctuation.parenthesis.begin.python, source.python
-?             : invalid.illegal.character.python, source.python
+?             : invalid.illegal.operator.python, source.python
 x             : source.python
 )             : punctuation.parenthesis.end.python, source.python
         foo   : source.python

Original file line number	Diff line number	Diff line change
`@@ -290,6 +290,9 @@ repository:`
`290`	`290`	`{`
`291`	`291`	`include: "#lambda"`
`292`	`292`	`}`
	`293`	`+ {`
	`294`	`+ include: "#illegal-operator"`
	`295`	`+ }`
`293`	`296`	`{`
`294`	`297`	`include: "#operator"`
`295`	`298`	`}`
`@@ -332,9 +335,6 @@ repository:`
`332`	`335`	`{`
`333`	`336`	`include: "#special-names"`
`334`	`337`	`}`
`335`		`- {`
`336`		`- include: "#illegal-operator"`
`337`		`- }`
`338`	`338`	`{`
`339`	`339`	`include: "#illegal-names"`
`340`	`340`	`}`
`@@ -1677,11 +1677,15 @@ repository:`
`1677`	`1677`	`"illegal-operator":`
`1678`	`1678`	`patterns: [`
`1679`	`1679`	`{`
`1680`		`- name: "invalid.illegal.character.python"`
	`1680`	`+ name: "invalid.illegal.operator.python"`
	`1681`	`+ match: "&&\|\\\|\\\|"`
	`1682`	`+ }`
	`1683`	`+ {`
	`1684`	`+ name: "invalid.illegal.operator.python"`
`1681`	`1685`	`match: "[?$]"`
`1682`	`1686`	`}`
`1683`	`1687`	`{`
`1684`		`- name: "invalid.illegal.character.python"`
	`1688`	`+ name: "invalid.illegal.operator.python"`
`1685`	`1689`	comment: "We don't want `!` to flash when we're typing `!=`"
`1686`	`1690`	`match: "!\\b"`
`1687`	`1691`	`}`