Reviewers: Sven,

Message:
Please take a look.
I didn't use the patch by the bug issuer btw.

Description:
Better error messages for d8's load() (execute from file).
BUG=v8:796


Please review this at http://codereview.chromium.org/7754018/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/d8.cc


Index: src/d8.cc
diff --git a/src/d8.cc b/src/d8.cc
index 93b383d9acda6c77560a7cf459613263db59d29c..1fdcc3e8146edb05a413dcf99fb04fc65faa17bd 100644
--- a/src/d8.cc
+++ b/src/d8.cc
@@ -271,7 +271,7 @@ Handle<Value> Shell::Load(const Arguments& args) {
     if (source.IsEmpty()) {
       return ThrowException(String::New("Error loading file"));
     }
-    if (!ExecuteString(source, String::New(*file), false, false)) {
+    if (!ExecuteString(source, String::New(*file), false, true)) {
       return ThrowException(String::New("Error executing file"));
     }
   }
@@ -439,6 +439,7 @@ void Shell::ReportException(v8::TryCatch* try_catch) {
       printf("%s\n", stack_trace_string);
     }
   }
+  printf("\n");
 }




--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to