Opened 12 years ago

Closed 12 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 12 years ago.
Patch to svn trunk that adds reposition threshold support

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by jk1joel

Attachment: repos_threshold.patch added

Patch to svn trunk that adds reposition threshold support

comment:1 Changed 12 years ago by jk1joel

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 Changed 12 years ago by danielk

Milestone: 0.210.22
Status: newinfoneeded_new

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

comment:3 Changed 12 years ago by stuartm

Resolution: invalid
Status: infoneeded_newclosed

Requested info not supplied after 30 days.

Note: See TracTickets for help on using tickets.