[mythtv-commits] Ticket #6889: New commercial detection method to flag Pre and post roll only
MythTV
mythtv at cvs.mythtv.org
Sat Aug 22 22:27:34 UTC 2009
#6889: New commercial detection method to flag Pre and post roll only
--------------------------------------+-------------------------------------
Reporter: Paul <Paul at formagio.net> | Owner: cpinkham
Type: enhancement | Status: assigned
Priority: minor | Milestone: 0.22
Component: MythTV - General | Version: head
Severity: low | Resolution:
Mlocked: 0 |
--------------------------------------+-------------------------------------
Comment(by cpinkham):
I don't see where you are waiting until after the pre-roll to being
flagging. In ::go(), requiredHeadStart is set to requiredBuffer which is
set to 120, you never add in preRoll.
Can you wrap your long lines at 80 columns by splitting strings and
putting function or QString args on the next line, etc.?
Myth's coding standard says that opening brackets for blocks of code go on
the next line, not on the if/while/for/function line, please correct these
as well.
Duplicated code here: (COMM_FRAME_SCENE_CHANGE | COMM_FRAME_BLANK |
COMM_FRAME_SCENE_CHANGE)
Please include a patch to libs/libmyth/programinfo.cpp to add this
detection method to the SkipTypeToString() and
GetPreferredSkipTypeCombinations() functions there so that the user would
not have to edit the database to change the default or channel-specific
flagging methods.
How accurate is this and what kind of pre/post roll did you use when
testing. It appears that you are only looking for the closest blank frame
or scene change frame to the pre/post roll times.
If you can correct these issues in an updated patch, I should be able to
get this in before the feature freeze on Monday.
--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6889#comment:2>
MythTV <http://www.mythtv.org/>
MythTV
More information about the mythtv-commits
mailing list