dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: RE: WPS implementation

RE: WPS implementation

From: Lee Marlow <>
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:

Index: wps.c
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;
+ // play previous in playlist
+ break;
+ //play next in playlist
+ mpeg_play(playlist_next(1));
+ //sleep(HZ);
+ //id3 = mpeg_current_track();
+ break;

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

irc: mecraw

-----Original Message-----
From: [] On
Behalf Of Daniel Stenberg
Sent: Monday, June 24, 2002 4:17 PM
Subject: Re: WPS implementation

On Mon, 24 Jun 2002 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
actual mp3
> plays.

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 =>
Received on 2002-06-26

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy