Rockbox mail archiveSubject: RE: WPS implementation
RE: WPS implementation
From: Lee Marlow <lmarlow_at_yahoo.com>
Date: Tue, 25 Jun 2002 18:41:18 -0600
Just thought I'd add my two cents (4 lines of code, in this case) for
WPS. The one thing that keeps me from using rockbox fulltime, besides
battery life, is being able to skip to the next track in a playlist.
So, here is my little patch for the JBR20:
RCS file: /cvsroot/rockbox/apps/wps.c,v
retrieving revision 1.9
diff -u -r1.9 wps.c
--- wps.c 25 Jun 2002 17:44:05 -0000 1.9
+++ wps.c 26 Jun 2002 00:28:43 -0000
_at__at_ -165,6 +165,18 _at__at_
global_settings.volume = 0;
+ case BUTTON_LEFT:
+ // play previous in playlist
+ case BUTTON_RIGHT:
+ //play next in playlist
+ //id3 = mpeg_current_track();
I hope that's the right format. This is my first time coding C and
contributing here. Unfortunately, I couldn't get the WPS to update with
the new track info, but I'll leave that as an exercise for the more
From: owner-rockbox_at_cool.haxx.se [mailto:owner-rockbox_at_cool.haxx.se] On
Behalf Of Daniel Stenberg
Sent: Monday, June 24, 2002 4:17 PM
Subject: Re: WPS implementation
On Mon, 24 Jun 2002 jerome_at_geekrox.com wrote:
> 1) playlists force back to the directory browse after the first track
> loads. I have yet to find which part of the code causes this.
Heh, yeah it looks a bit silly! ;-)
> 2) I still need to hook into the "on" button or something to allow use
> toggle of WPS.
On the Recorder, one of the functions might be a good choice.
> 3) Despite the "QUICK hack" comment, I still used peek_next_track to
> determine display time. Thus the screen gets modified before the
We can't use peek_next_track() for that information. It is used for mp3
buffering, not for actual playback info.
> The first two are easy enough, but the third brings up questions of
> future for WPS. A couple things I've been thinking....
> 1) Improve the interface between mpeg & peek_next_track. Since it is
> "QUICK hack," a "fix" is required anyway.
The interface here is that the mpeg thread is supposed to read the id3
from the song it plays. The WPS code could then read the information
That's the plan at least. I haven't yet tried to add this myself.
> 2) Insitute WPS into it's own thread to take events when to display or
> way to the another mode (like dirbrowse). Possibly this fits into
> 1, but is a bit more involved.
I'm not sure this needs to be a separate thread. In fact, having it as a
separate thread from the UI thread might cause more problems that it
> So any help is appreciated. I haven't submitted any code, but would
> willing to tar ball anyone on the list who requests.
I prefer a diff -u output! ;-)
Uh, btw, *great* work!
-- Daniel Stenberg -- Rocking the box => http://bjorn.haxx.se/rockbox/Received on 2002-06-26