Reviewers: Vitaly Repeshko,

Description:
Minor cosmetic changes.


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

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

Affected files:
  M src/messages.cc
  M test/cctest/test-api.cc


Index: src/messages.cc
diff --git a/src/messages.cc b/src/messages.cc
index abc25377851b5c4a76a404b5519edf75adc70226..0fdbbbb79ed9ef5c960dfb5c49c5f1de21c4008b 100644
--- a/src/messages.cc
+++ b/src/messages.cc
@@ -131,7 +131,7 @@ void MessageHandler::ReportMessage(Isolate* isolate,
       Handle<Object> callback_data(listener.get(1));
       {
         // Do not allow exceptions to propagate.
-        v8::TryCatch tryCatch;
+        v8::TryCatch try_catch;
         callback(api_message_obj, v8::Utils::ToLocal(callback_data));
       }
       if (isolate->has_scheduled_exception()) {
Index: test/cctest/test-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index d7621d12cf2ff1af6c02c36abd4c626b418eb612..28a9be159937b9f1399efa6f7de5e9734751ba55 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -8711,6 +8711,8 @@ THREADED_TEST(ExceptionsDoNotPropagatePastTryCatch) {
     if (callback != NULL) {
       V8::AddMessageListener(callback);
     }
+    // Some small number to control number of times message handler should
+    // throw an exception.
     call_depth = 5;
     ExpectFalse(
         "var thrown = false;\n"


--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev

Reply via email to