<div dir="ltr">LGTM for all 5 patches<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 17, 2015 at 10:14 AM, Thomas Guillem <span dir="ltr"><<a href="mailto:thomas@gllm.fr" target="_blank">thomas@gllm.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This reverts commit bcc71ef19234269f39085e5841eb006c96b22dd6.<br>
<br>
It's better to remove event handler and clear events.<br>
---<br>
 vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java b/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java<br>
index 146a712..2e8b695 100644<br>
--- a/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java<br>
+++ b/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java<br>
@@ -1292,7 +1292,7 @@ public class VideoPlayerActivity extends ActionBarActivity implements IVideoPlay<br>
         @Override<br>
         public void handleMessage(Message msg) {<br>
             VideoPlayerActivity activity = getOwner();<br>
-            if(activity == null || activity.isFinishing()) return;<br>
+            if(activity == null) return;<br>
             // Do not handle events if we are leaving the VideoPlayerActivity<br>
             if (activity.mSwitchingView) return;<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.1.3<br>
<br>
_______________________________________________<br>
Android mailing list<br>
<a href="mailto:Android@videolan.org">Android@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/android" target="_blank">https://mailman.videolan.org/listinfo/android</a><br>
</font></span></blockquote></div><br></div>