Regex literals with leading spaces

[tobyh]

I think this bug was introduced in 0.95

$ coffee -bpe "r = / x/"
Error: Parse error on line 1: Unexpected 'MATH'

[tobyh]

This has come up previously. Is it a regression, or a reversion to the old syntax rule, or am I imagining that this worked in 0.94?

[StanAngeloff]

Bisecting seems to think cdd033ffb0b88eb6f00e is the first bad commit:

(stan) [coffee-script] % git checkout ^cdd033ffb0b88eb6f00e
(stan) [coffee-script] % bin/coffee -ep 'r = / x/'
(function() {
  var r;
  r = / x/;
}).call(this);
(stan) [coffee-script] % git checkout cdd033ffb0b88eb6f00e
Previous HEAD position was 493fa7d... heregex: now allows bare slashes and empty body
HEAD is now at cdd033f... disabled interpolations in normal regexes
(stan) [coffee-script] % bin/coffee -ep 'r = / x/'
Error: In console, Parse error on line 1: Unexpected 'MATH'
    at Object.parseError (/cygdrive/d/Workspace/public/coffee-script/lib/parser.js:513:11)
    at Object.parse (/cygdrive/d/Workspace/public/coffee-script/lib/parser.js:585:32)
    at Object.compile (/cygdrive/d/Workspace/public/coffee-script/lib/coffee-script.js:22:22)
    at /cygdrive/d/Workspace/public/coffee-script/lib/command.js:116:33
    at Object.run (/cygdrive/d/Workspace/public/coffee-script/lib/command.js:33:14)
    at Object.<anonymous> (/cygdrive/d/Workspace/public/coffee-script/bin/coffee:8:27)
    at Module._compile (node.js:462:23)
    at Module._loadScriptSync (node.js:469:10)
    at Module.loadSync (node.js:338:12)
    at Object.runMain (node.js:522:24)

Looking at the commit contents it was intended. However, it seems like hereregexes strip leading whitespace so /// x/// is not an option either. Anyone with a link to the discussion that axed this?

[satyr]

This was killed because of inconsistency wrt parenless calls.
r = / x/ # worked
fun / x/ # worked not

Here are some workarounds for the case:

• /\ x/
• /// \x20 x ///

[tobyh]

I think this might be a case where the less consistent behaviour was also less likely to cause surprises, but the change is understandable. Thanks for the answer.

[akidee]

fun / x/  # worked not

So CS interpreted "/" as function name here?

[satyr]

"/" as function name here?

A division operator, rather.

[akidee]

Should be added to the doc.