Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Commit

Permalink
Remove TODO to use v8 as parser
Browse files Browse the repository at this point in the history
It seems that this has been asked for in the past and the V8 team
justified why they do not want these apis to be exposed. See Andreas
Rossberg's answer here:
https://groups.google.com/forum/#!topic/v8-users/_WracRX9BTQ
  • Loading branch information
Matt Loring committed Jan 19, 2016
1 parent 3550675 commit d667fa6
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions lib/state.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ function evaluate(expression, frame) {
}

// Now actually ask V8 to evaluate the expression
// TODO: it would be really nice if V8 gave us an API for this
// so that we don't have to include an external parser
try {
var mirror = frame.evaluate(expression);
return {
Expand Down

0 comments on commit d667fa6

Please sign in to comment.