From ff91e514511194ad22aa47bfaa26fee39dac46a5 Mon Sep 17 00:00:00 2001 From: Christopher Eby Date: Sun, 21 Mar 2010 16:23:52 -0500 Subject: [PATCH] Just look for the handler on startup instead of a custom variable We depend on mHandler to begin with, so the variable could cause trouble --- src/org/kreed/vanilla/PlaybackService.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/org/kreed/vanilla/PlaybackService.java b/src/org/kreed/vanilla/PlaybackService.java index f9f6687f..30f4c080 100644 --- a/src/org/kreed/vanilla/PlaybackService.java +++ b/src/org/kreed/vanilla/PlaybackService.java @@ -65,7 +65,6 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On public static final int STATE_NO_MEDIA = 1; public static final int STATE_PLAYING = 2; - private boolean mStarted; private int mPendingGo = -1; public IPlaybackService.Stub mBinder = new IPlaybackService.Stub() { @@ -138,7 +137,7 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On private void go(int delta) { - if (!mStarted) { + if (mHandler == null) { Toast.makeText(this, R.string.starting, Toast.LENGTH_SHORT).show(); mPendingGo = delta; return; @@ -167,8 +166,6 @@ public class PlaybackService extends Service implements Runnable, MediaPlayer.On if (mHandler != null) mHandler.sendMessage(mHandler.obtainMessage(DO_ITEM, intent)); - - mStarted = true; } @Override