summaryrefslogtreecommitdiffstats
path: root/core/java/android/app
diff options
context:
space:
mode:
authorAndroid (Google) Code Review <android-gerrit@google.com>2009-06-16 13:14:09 -0700
committerAndroid (Google) Code Review <android-gerrit@google.com>2009-06-16 13:14:09 -0700
commit5633034f79bfa67c4219f16fd6d23e962e800be2 (patch)
tree8566caee5e5acf3a71124fff06596b3b85456fbd /core/java/android/app
parenta6061e043cb6056c303206c2c9870ae3758861f1 (diff)
parent12172baa6b72a8996b3f043f18067c9b43ed3bc8 (diff)
downloadframeworks_base-5633034f79bfa67c4219f16fd6d23e962e800be2.zip
frameworks_base-5633034f79bfa67c4219f16fd6d23e962e800be2.tar.gz
frameworks_base-5633034f79bfa67c4219f16fd6d23e962e800be2.tar.bz2
Merge change 4115 into donut
* changes: * component can be null after destory? This was causing NPE in catch block, which hides the actual exception.
Diffstat (limited to 'core/java/android/app')
-rw-r--r--core/java/android/app/ActivityThread.java7
1 files changed, 4 insertions, 3 deletions
diff --git a/core/java/android/app/ActivityThread.java b/core/java/android/app/ActivityThread.java
index d08fc11..2fe6471 100644
--- a/core/java/android/app/ActivityThread.java
+++ b/core/java/android/app/ActivityThread.java
@@ -31,6 +31,7 @@ import android.content.pm.InstrumentationInfo;
import android.content.pm.PackageManager;
import android.content.pm.ProviderInfo;
import android.content.pm.ServiceInfo;
+import android.content.pm.PackageParser.Component;
import android.content.res.AssetManager;
import android.content.res.Configuration;
import android.content.res.Resources;
@@ -3286,10 +3287,10 @@ public final class ActivityThread {
throw e;
} catch (Exception e) {
if (!mInstrumentation.onException(r.activity, e)) {
+ ComponentName component = r.intent.getComponent();
+ String name = component == null ? "[Unknown]" : component.toShortString();
throw new RuntimeException(
- "Unable to destroy activity "
- + r.intent.getComponent().toShortString()
- + ": " + e.toString(), e);
+ "Unable to destroy activity " + name + ": " + e.toString(), e);
}
}
}