This repository has been archived by the owner. It is now read-only.

Partial fix for issue 3452, syntax errors in vm.runInThisContext don't display proper line numbers #3570

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@itaylor

itaylor commented Jun 28, 2012

This pull request (joyent#3502) was already open and had been reviewed, but in the process of cleaning up my commit history, I took some action which inadvertently closed my old pull request. Sorry about that.

@Nodejs-Jenkins

This comment has been minimized.

Show comment Hide comment
@Nodejs-Jenkins

Nodejs-Jenkins Mar 13, 2013

Can one of the admins verify this patch?

Can one of the admins verify this patch?

bnoordhuis added a commit to bnoordhuis/node that referenced this pull request Jul 29, 2013

@tjfontaine

This comment has been minimized.

Show comment Hide comment
@tjfontaine

tjfontaine Feb 18, 2014

Hi, thanks this is fixed in the current builds

Hi, thanks this is fixed in the current builds

@tjfontaine tjfontaine closed this Feb 18, 2014

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.