Skip to content

Commit

Permalink
Program editor: don't seek too far when stepping to the next keyframe.
Browse files Browse the repository at this point in the history
Fixes #10389.  When stepping to the next keyframe to the right, some
old code causes it to skip over the next keyframe if it is too close.
This code seems unnecessary, but it is left there for now (commented
out) in case a valid reason is discovered.
  • Loading branch information
stichnot committed Feb 29, 2012
1 parent 97da8c8 commit a6b1e3a
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions mythtv/libs/libmythtv/mythplayer.cpp
Expand Up @@ -3975,12 +3975,19 @@ void MythPlayer::HandleArbSeek(bool right)
{
if (right)
{
#if 0
// 2012-02-29. This logic doesn't seem to make sense for
// the current code. Clean it out later if no one raises
// an issue with the replacement code. Refs #10389.

// editKeyFrameDist is a workaround for when keyframe distance
// is set to one, and keyframe detection is disabled because
// the position map uses MARK_GOP_BYFRAME. (see DecoderBase)
float editKeyFrameDist = keyframedist <= 2 ? 18 : keyframedist;

DoFastForward((long long)(editKeyFrameDist * 1.1), true, false);
#endif // 0
DoFastForward(2, true, false);
}
else
{
Expand Down

0 comments on commit a6b1e3a

Please sign in to comment.