Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

[dvdplayer] remove old disabled looping still frame handling code

Disabled code that should be written differently if we
every need it again.
  • Loading branch information...
commit d6a631b7765e9f3cd4723980e72f03383c00473b 1 parent 3277107
@elupus authored
Showing with 0 additions and 37 deletions.
  1. +0 −37 xbmc/cores/dvdplayer/DVDPlayer.cpp
View
37 xbmc/cores/dvdplayer/DVDPlayer.cpp
@@ -1612,43 +1612,6 @@ void CDVDPlayer::CheckContinuity(CCurrentStream& current, DemuxPacket* pPacket)
if( pPacket->dts == DVD_NOPTS_VALUE || current.dts == DVD_NOPTS_VALUE)
return;
-#if 0
- // these checks seem to cause more harm, than good
- // looping stillframes are not common in normal files
- // and a better fix for this behaviour would be to
- // correct the timestamps with some offset
-
- if (current.type == STREAM_VIDEO
- && m_CurrentAudio.dts != DVD_NOPTS_VALUE
- && m_CurrentVideo.dts != DVD_NOPTS_VALUE)
- {
- /* check for looping stillframes on non dvd's, dvd's will be detected by long duration check later */
- if( m_pInputStream && !m_pInputStream->IsStreamType(DVDSTREAM_TYPE_DVD))
- {
- /* special case for looping stillframes THX test discs*/
- /* only affect playback when not from dvd */
- if( (m_CurrentAudio.dts > m_CurrentVideo.dts + DVD_MSEC_TO_TIME(200))
- && (m_CurrentVideo.dts == pPacket->dts) )
- {
- CLog::Log(LOGDEBUG, "CDVDPlayer::CheckContinuity - Detected looping stillframe");
- SynchronizePlayers(SYNCSOURCE_VIDEO);
- return;
- }
- }
-
- /* if we haven't received video for a while, but we have been */
- /* getting audio much more recently, make sure video wait's */
- /* this occurs especially on thx test disc */
- if( (pPacket->dts > m_CurrentVideo.dts + DVD_MSEC_TO_TIME(200))
- && (pPacket->dts < m_CurrentAudio.dts + DVD_MSEC_TO_TIME(50)) )
- {
- CLog::Log(LOGDEBUG, "CDVDPlayer::CheckContinuity - Potential long duration frame");
- SynchronizePlayers(SYNCSOURCE_VIDEO);
- return;
- }
- }
-#endif
-
double mindts = DVD_NOPTS_VALUE, maxdts = DVD_NOPTS_VALUE;
UpdateLimits(mindts, maxdts, m_CurrentAudio.dts);
UpdateLimits(mindts, maxdts, m_CurrentVideo.dts);
Please sign in to comment.
Something went wrong with that request. Please try again.