From 4ba297f3ed2def2845edb18723e43379984bc9ff Mon Sep 17 00:00:00 2001 From: Marco Nelissen Date: Fri, 26 Feb 2010 09:30:20 -0800 Subject: [PATCH] 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 --- media/java/android/media/AsyncPlayer.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/media/java/android/media/AsyncPlayer.java b/media/java/android/media/AsyncPlayer.java index e1e09b96acc06..09aec2e6a647f 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); } }