summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Kendall <mkendall@mythtv.org>2011-03-15 06:31:11 (GMT)
committer Mark Kendall <mkendall@mythtv.org>2011-03-15 06:31:11 (GMT)
commit5ea8d18065886e262f0b2507ea7e99a0eb49bcc7 (patch)
tree7647f46b292d1c00a2f7cf831ebe62a4b0431c15
parent9ce70aed37ae2887fa56b0eaf8cb37b5e10373d4 (diff)
MythPlayer:: Remove some unnecessary decoder locking.
The decoder_change_lock is used by the decoder thread to prevent the main thread from deleting/recreating the decoder object while it is in use. It does not need to be used from the main thread.
-rw-r--r--mythtv/libs/libmythtv/mythplayer.cpp26
1 files changed, 8 insertions, 18 deletions
diff --git a/mythtv/libs/libmythtv/mythplayer.cpp b/mythtv/libs/libmythtv/mythplayer.cpp
index 728209f..ef2224b 100644
--- a/mythtv/libs/libmythtv/mythplayer.cpp
+++ b/mythtv/libs/libmythtv/mythplayer.cpp
@@ -303,8 +303,6 @@ MythPlayer::~MythPlayer(void)
void MythPlayer::SetWatchingRecording(bool mode)
{
- QMutexLocker locker(&decoder_change_lock);
-
watchingrecording = mode;
if (decoder)
decoder->setWatchingRecording(mode);
@@ -352,7 +350,6 @@ bool MythPlayer::Pause(void)
PauseBuffer();
allpaused = decoderPaused && videoPaused && bufferPaused;
{
- QMutexLocker locker(&decoder_change_lock);
if (using_null_videoout && decoder)
decoder->UpdateFramesPlayed();
else if (videoOutput && !using_null_videoout)
@@ -1285,7 +1282,6 @@ void MythPlayer::ResetCaptions(void)
}
}
-// caller has decoder_changed_lock
void MythPlayer::DisableCaptions(uint mode, bool osd_msg)
{
textDisplayMode &= ~mode;
@@ -1326,7 +1322,6 @@ void MythPlayer::DisableCaptions(uint mode, bool osd_msg)
}
}
-// caller has decoder_changed_lock
void MythPlayer::EnableCaptions(uint mode, bool osd_msg)
{
QMutexLocker locker(&osdLock);
@@ -1422,7 +1417,6 @@ void MythPlayer::SetCaptionsEnabled(bool enable, bool osd_msg)
QStringList MythPlayer::GetTracks(uint type)
{
- QMutexLocker locker(&decoder_change_lock);
if (decoder)
return decoder->GetTracks(type);
return QStringList();
@@ -1431,7 +1425,6 @@ QStringList MythPlayer::GetTracks(uint type)
int MythPlayer::SetTrack(uint type, int trackNo)
{
int ret = -1;
- QMutexLocker locker(&decoder_change_lock);
if (!decoder)
return ret;
@@ -1487,7 +1480,6 @@ void MythPlayer::EnableSubtitles(bool enable)
int MythPlayer::GetTrack(uint type)
{
- QMutexLocker locker(&decoder_change_lock);
if (decoder)
return decoder->GetTrack(type);
return -1;
@@ -1495,23 +1487,21 @@ int MythPlayer::GetTrack(uint type)
int MythPlayer::ChangeTrack(uint type, int dir)
{
- QMutexLocker locker(&decoder_change_lock);
- if (decoder)
+ if (!decoder)
+ return -1;
+
+ int retval = decoder->ChangeTrack(type, dir);
+ if (retval >= 0)
{
- int retval = decoder->ChangeTrack(type, dir);
- if (retval >= 0)
- {
- SetOSDMessage(decoder->GetTrackDesc(type, GetTrack(type)),
- kOSDTimeout_Med);
- return retval;
- }
+ SetOSDMessage(decoder->GetTrackDesc(type, GetTrack(type)),
+ kOSDTimeout_Med);
+ return retval;
}
return -1;
}
void MythPlayer::ChangeCaptionTrack(int dir)
{
- QMutexLocker locker(&decoder_change_lock);
if (!decoder || (dir < 0))
return;