Remove TODO to use v8 as parser

Matt Loring · Matt Loring · commit d667fa646b52 · 2016-01-19T10:59:25.000-08:00
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
diff --git a/lib/state.js b/lib/state.js
@@ -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 {

Original file line number	Diff line number	Diff line change
`@@ -52,8 +52,6 @@ function evaluate(expression, frame) {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`// Now actually ask V8 to evaluate the expression`
`55`		`- // TODO: it would be really nice if V8 gave us an API for this`
`56`		`- // so that we don't have to include an external parser`
`57`	`55`	`try {`
`58`	`56`	`var mirror = frame.evaluate(expression);`
`59`	`57`	`return {`