[mythtv-commits] Ticket #9997: patch: Fix commercial flagging to return consistent results

MythTV noreply at mythtv.org
Fri Aug 19 20:34:25 UTC 2011


#9997: patch: Fix commercial flagging to return consistent results
---------------------------------------+----------------------
     Reporter:  kevin@…                |      Owner:  cpinkham
         Type:  Bug Report - General   |     Status:  new
     Priority:  minor                  |  Milestone:  unknown
    Component:  MythTV - Mythcommflag  |    Version:  0.24.1
     Severity:  medium                 |   Keywords:
Ticket locked:  0                      |
---------------------------------------+----------------------
 Running mythcommflag on a recording, and comparing the results of
 --getskiplist after rerunning the command several times would result in
 inconsistent results.  One would expect to get the same results every
 time.  Also, the results of commercial flagging were usually rather poor
 (for me).  This seemed to be a regression in 0.24.

 I believe the problem was with the implementation of
 MythPlayer::GetRawVideoFrame.  At the end of the function, it would return
 the result of GetLastDecodedFrame.  I believe this to be incorrect.  When
 printing the frame numbers of the frames returned, sometimes I would get
 the same frame number returned multiple times, and other times it would
 skip a frame or two (or more).  Basically, it's returning the frame from
 the tail of the queue, when it should be returning the frame from the head
 of the queue.

 My fix is to call StartDisplayingFrame, followed by GetLastShownFrame.
 This returns the frame from the head of the queue, and printing the frame
 numbers always results in a nice even output of frame numbers, no
 duplicates, no skips.

 Now my commercial flagging is much improved, and rerunning mythcommflag
 multiple times on a recording results in the same results each time.

 -- Kevin

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/9997>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center


More information about the mythtv-commits mailing list