commit 710a0cbb4b0af03ce8651ed7936437c73250045e
author Elliott Hughes <enh@google.com> Tue Aug 16 14:32:37 2011 -0700
committer Elliott Hughes <enh@google.com> Tue Aug 16 14:32:37 2011 -0700
tree 161a43b45307333ffba2b9751ba4ed0a5b16e72c
parent a5b897eae4b6f9f9608faa9eada7ddf42bf1bfd2

Extra tests (and a bug fix).

I've also added a trailing " " to the CHECK/UNIMPLEMENTED macros'
output, to make it more natural for callers to add their own detail.

Change-Id: I59bc51ac54aaff93f37a6b35082301e19db91304

src/thread.cc

diff --git a/src/thread.cc b/src/thread.cc
index 51b72ce..609ae41 100644
--- a/src/thread.cc
+++ b/src/thread.cc
@@ -147,11 +147,11 @@
   // TODO: what if there's already a pending exception?
   // TODO: support the other constructors.
   Method* ctor = exception_class->FindDirectMethod("<init>", "(Ljava/lang/String;)V");
+  CHECK(ctor != NULL);
 
   // TODO: need to *call* the constructor!
   UNIMPLEMENTED(WARNING) << "can't call "
-                         << exception_class->GetDescriptor() << "."
-                         << ctor->GetDescriptor() << " "
+                         << exception_class->GetDescriptor() << ".<init> "
                          << "\"" << msg << "\"";
 
   thread->SetException(exception);