commit 1fbf63abf41ca1fb9ae2c5d82088e74475279414
author Andreas Huber <andih@google.com> Tue Mar 16 14:31:52 2010 -0700
committer Andreas Huber <andih@google.com> Tue Mar 16 14:31:52 2010 -0700
tree 594ef71a7c44a876980721d4a56ea9c49f64993c
parent 4f33d80056dc8a185176085a211bd31f143686c4

"isPlaying" should never block, acquiring the mutex here is unnecessary, rather than investing in a separate mutex guarding just "mFlags" we accept returning "true" from isPlaying one too many times.

Change-Id: I597577cbb3ad1afd3cb3ca91979ef24cc1a44658
related-to-bug: 2518319

media/libstagefright/AwesomePlayer.cpp

diff --git a/media/libstagefright/AwesomePlayer.cpp b/media/libstagefright/AwesomePlayer.cpp
index 10b7be34..164df72 100644
--- a/media/libstagefright/AwesomePlayer.cpp
+++ b/media/libstagefright/AwesomePlayer.cpp
@@ -624,8 +624,6 @@
 }
 
 bool AwesomePlayer::isPlaying() const {
-    Mutex::Autolock autoLock(mLock);
-
     return mFlags & PLAYING;
 }