diff options
author | Marco Nelissen <marcone@google.com> | 2010-02-26 09:30:20 -0800 |
---|---|---|
committer | Marco Nelissen <marcone@google.com> | 2010-02-26 09:30:20 -0800 |
commit | 4ba297f3ed2def2845edb18723e43379984bc9ff (patch) | |
tree | 0dc8653be18adb38bec23bf76606c0ae4661e5df /media | |
parent | e379710cc5914c7ed10e9236079ec7f02c99ee4e (diff) | |
download | frameworks_base-4ba297f3ed2def2845edb18723e43379984bc9ff.zip frameworks_base-4ba297f3ed2def2845edb18723e43379984bc9ff.tar.gz frameworks_base-4ba297f3ed2def2845edb18723e43379984bc9ff.tar.bz2 |
Make AsyncPlayer.startSound catch all exceptions instead of only a few.
Two reasons for this:
- it's used in the system process
- it's run in a separate thread, so the user of AsyncPlayer cannot catch these exceptions
Diffstat (limited to 'media')
-rw-r--r-- | media/java/android/media/AsyncPlayer.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/media/java/android/media/AsyncPlayer.java b/media/java/android/media/AsyncPlayer.java index e1e09b9..09aec2e 100644 --- a/media/java/android/media/AsyncPlayer.java +++ b/media/java/android/media/AsyncPlayer.java @@ -72,10 +72,8 @@ public class AsyncPlayer { Log.w(mTag, "Notification sound delayed by " + delay + "msecs"); } } - catch (IOException e) { + catch (Exception e) { Log.w(mTag, "error loading sound for " + cmd.uri, e); - } catch (IllegalStateException e) { - Log.w(mTag, "IllegalStateException (content provider died?) " + cmd.uri, e); } } |