atom / language-coffee-script

CoffeeScript support in Atom
Other
59 stars 56 forks source link

Add proper boundary-checking for and=, or= #157

Closed winstliu closed 7 years ago

winstliu commented 7 years ago

Requirements

Description of the Change

and= and or= were being lumped with the other compound assignment operators, which since they're symbols do not require any whitespace around the operator. and= and or= however do require whitespace since they're English. Therefore, extract those two into their own pattern with proper boundary checking (identifier + spaces, or no identifier before the operator).

Alternate Designs

None.

Benefits

operand=, editor= will be tokenized correctly.

Possible Drawbacks

None.

Applicable Issues

Fixes #152