Issue #403 resolved

Incorrect RegExp detection in JavaScript

Anonymous created an issue

In !JavaScript, a single forward slash following one of {{{ [({=,:;!%^&*|?~+- }}} starts a regular expression. Spaces are not required. For example, this is valid syntax: {{{ if (!/regexp/.test(foo)) }}} but this is not recognized by Pygments.

See more examples at

Reported by Pumbaa80

Comments (10)

  1. Anonymous

    I had a look at [] and I'm afraid there are [ some more issues] with !JavascriptLexer.

    First, according to [ the JavaScript spec], this should be the right way to match operators and punctuators:

    (r'=|++|--|~|&&|\\?|:|\\|\\||\\\\|(<<|>>>?|==?|!=?|[<>+-*%&\\|\\^/])=?', Operator)
    (r'[{}()\\[\\].;,]', Punctuation)

    Second, character classes need a special treatment... (use several states?)

    Once this is fixed, the detection of a regexp literal's end (cf. lines 47 to 49 in can be simplified/fixed. This should sufficiently replace the three lines:

    ...   /([gim]+\\b|\\B)', String.Regex)

    !ActionScriptLexer and !ActionScript3Lexer might need a fix, too (?).

  2. Martin Bodin


    I’m opening this bug again (although the versions of Pygments changed a lot in the meantime, so maybe I should have opened a new bug). There is a mistaking coloring for the following code: It is indeed parsed as if it was the following one:

    In Javascript, semicolons are only added if the absence of the semicolon leads to a syntax error. The correct parsing for should be {{{ n = < < 1 / 1 > * "<string>" > // <comment> }}} instead of {{{ < n = 1 > < /<regexp>/.test (n + '<string>') }}} as it is right now.

    I’ve had this problem by switching from Pygments 1.6 to 2.0.2: did something important changed in the meantime?

    Sincerely, Martin.

    Edit: I’ve posted this issue in the meantime, as it is probably just another bug:

  3. Log in to comment