atom / language-coffee-script

CoffeeScript support in Atom
Other
59 stars 56 forks source link

Match regexes containing multiple escaped forward slashes #139

Closed winstliu closed 7 years ago

winstliu commented 7 years ago

Requirements

Description of the Change

The regex regex just keeps getting regexier. Who doesn't like a negative lookahead in a lookahead? Now a begin/end match in an attempt to avoid some very ugly lookbehinds that may not even be possible.

Alternate Designs

Attempt to use lookbehinds.

Benefits

Maybe this will be the last regex fix.

Possible Drawbacks

Maybe this won't be the last regex fix.

Applicable Issues

Fixes #127

/cc @Alhadis