Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Fix MythPlayer::GetRawVideoFrame() for mythcommflag.

Fix MythPlayer::GetRawVideoFrame() to return the correct frame number.
This change fixes mythcommflag so that it will return consistent results
across multiple runs.  It should also improve commercial detection
accuracy, as the frame numbers identified by the different methods will
actually be precise--allowing for better comparing the results of the
different methods.

Although this commit has my name for the signed-off line, I didn't do
much.  Thanks to Kevin Ross for the debugging and the patch, to Taylor
Ralph and Mark Kendall for actually reviewing the patch, and to all the
users and devs who ran with the patch to test it (and saw
greatly-improved commercial flagging results).

Fixes #9997.

Signed-off-by: Michael T. Dean <mdean@mythtv.org>
(cherry picked from commit f135d5d)
  • Loading branch information...
commit cb01f3079a0eadf430178fefd5c2341235152fd1 1 parent a24ecbf
@k-ross k-ross authored sphery committed
Showing with 2 additions and 1 deletion.
  1. +2 −1  mythtv/libs/libmythtv/mythplayer.cpp
View
3  mythtv/libs/libmythtv/mythplayer.cpp
@@ -4089,7 +4089,8 @@ VideoFrame* MythPlayer::GetRawVideoFrame(long long frameNumber)
VERBOSE(VB_PLAYBACK, LOC + QString("Waited 100ms for video frame"));
}
- return videoOutput->GetLastDecodedFrame();
+ videoOutput->StartDisplayingFrame();
+ return videoOutput->GetLastShownFrame();
}
QString MythPlayer::GetEncodingType(void) const
Please sign in to comment.
Something went wrong with that request. Please try again.