Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

methods called delete kill the parser #72

Open
jaclar opened this issue Oct 23, 2012 · 1 comment
Open

methods called delete kill the parser #72

jaclar opened this issue Oct 23, 2012 · 1 comment

Comments

@jaclar
Copy link

jaclar commented Oct 23, 2012

Consider the following piece of javascript, which could be quiet common in an express.js node application:

function router(app) {
    app.post("/some/route", function (req, res, next) {
        next();
    });
    // this will break the parser:
    app.delete("/some/route", function (req,res, next) {
        next();
    });
}

The second route will break the syntax highlighter (and in general the javascript parser) because of the delete method.

@tamzinblake
Copy link
Owner

Yeah, the parser is old - reserved words as property names were disallowed until recently. Not sure if this is something to fix or wait till the new version with a new parser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants