Bug 8156 - Can't invoke FWD/REW speed multipiers
: Can't invoke FWD/REW speed multipiers
Status: RESOLVED INVALID
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: 7.1
: PC Windows XP
: -- normal (vote)
: 7.x
Assigned To: Alan Young
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-05-14 14:52 UTC by Philip Meyer
Modified: 2009-07-31 10:21 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Meyer 2008-05-14 14:52:11 UTC
The latest version of the fast-forward implementation doesn't seem to have the FWD/REW multiplier and audio feedback mechanism.

The version I previously had was really good, supporting both the old fast-forward multiplier mechanism and the new song scanner mechanism.

Press and hold FWD takes me to the song scanner, and then press and hold FWD moves the progress bar, rather than increase the speed multiplier.  i.e. it does the same as up/down buttons.

Is that accidental?  I saw a poll thread a while ago, voting for whether REW show slow down acceleration or immediately go backwards, but now it seems like the whole mechanism has totally dissapeared.

I notice the following errors being reported in the log:

[22:45:31.0686] Slim::Hardware::IR::executeButton (1031) Button [fwd.hold_release] with irCode: [undef] not implemented in mode: [Slim::Plugin::SongScanner::Plugin]
[22:45:38.6144] Slim::Hardware::IR::executeButton (1031) Button [fwd] with irCode: [undef] not implemented in mode: [Slim::Plugin::SongScanner::Plugin]
[22:45:40.8693] Slim::Hardware::IR::executeButton (1031) Button [fwd.hold_release] with irCode: [undef] not implemented in mode: [Slim::Plugin::SongScanner::Plugin]
Comment 1 Alan Young 2008-06-04 08:02:26 UTC
The switch to remove accelerated audio from SongScanner is a design decision. I know that you consider this to be a mistake, as we have discussed elsewhere.

In SC7.1 it would still be possible to use accelerated audio in SongScanner by switching a couple of lines in the plugin code. Look for the bindings for 'jump_fwd' and 'jump_rew' in %functions and swap the comments on the _scan and _jump calls. But with SC7.2 the current plan is to remove this support completely.

I'll look to getting the erroneous log warnings fixed.
Comment 2 KDF 2008-06-09 09:37:03 UTC
these warnings are perfectly valid if fwd.hold_release and fwd have no function in the plugin directly.  As the plugin uses INPUT.Bar, it is expected that most functionality if from there, not the plugin's mode. fwd.* is set up as a passback for INPUT.Bar so any use of these buttons will check the parent mode for any custom function.  Is there an intended function for 'fwd' and fwd.hold_release? 


simply setting the log level to ERROR will make them go away :)
set to INFO and you'll see more messages reporting the lack of an irCode definition.

If there is no function needed, they can be set to 'dead' like fwd.hold

Comment 3 Chris Owens 2009-07-31 10:21:29 UTC
Reduce number of active targets for SC