Opened 19 years ago

Closed 18 years ago

#4227 closed enhancement (invalid)

FF/Rew Reposition Threshold Support

Reported by: jk1joel Owned by: Isaac Richards
Priority: minor Milestone: 0.22
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This patch adds a threshold for the FF/Rew reposition feature. The original feature, in case you are not familiar with it, makes it so after fast forwing or rewinding in sticky mode, that the current position is repositioned back based upon how fast the user is fast forwarding. For example, if it is set to 1 second (the default), then when exiting fast forward in 3x, it will jump back 3 seconds, when exiting fast forward in 10x, it will jump back 10 seconds, and so on.

This patch, FF/Rew Reposition Threshold, makes it so that for slow FF/Rewind (below the threshold), no reposition jump takes place after leaving FF/Rew. The idea is that when fast forwarding slowly (like, say at 3x), the jump back may not be needed at all, but when fast forwarding quickly (like at 10x) it is. So by setting the threshold to 5, for example, any FF/Rew speed greater than or equal to 5X will trigger the reposition, but any FF/Rew speed less than 5X will not.

For the naysayers who say that this feature is not needed, this is how the TiVo does it, albeit without allowing for configuration. The slowest FF/Rew settings provide no reposition. So all of us ex-TiVo users will probably agree that this feature is nice to have. And it is set to default to 0, meaning that if repositioning is enabled, it is enabled for all speeds.

I have mine set to 5, and I'm very happy with it.

Attachments (1)

repos_threshold.patch (3.2 KB ) - added by jk1joel 19 years ago.
Patch to svn trunk that adds reposition threshold support

Download all attachments as: .zip

Change History (4)

by jk1joel, 19 years ago

Attachment: repos_threshold.patch added

Patch to svn trunk that adds reposition threshold support

comment:1 by jk1joel, 19 years ago

Based upon feedback from #mythtv, I am re-working this idea as a database-only setting, with no UI for configuration. This allows for customization for power users without overly cluttering the config UI. An updated patch will follow soon

comment:2 by danielk, 19 years ago

Milestone: 0.210.22
Status: newinfoneeded_new

Please select the provide info radiobox when you have the updated patch ready...

comment:3 by stuartm, 18 years ago

Resolution: invalid
Status: infoneeded_newclosed

Requested info not supplied after 30 days.

Note: See TracTickets for help on using tickets.