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

Rockbox mail archive

Subject: Re: WPS tags to display next-song info

Re: WPS tags to display next-song info

From: Daniel Stenberg <>
Date: Fri, 16 Jan 2004 08:43:23 +0100 (CET)

On Thu, 15 Jan 2004, Hardeep Sidhu wrote:

> I'm guessing that most users will be primarily interested in the next track
> name. This value can be retrieved even if the mpeg thread has not yet
> loaded the track by calling playlist_peek(1). So, how about special casing
> track name and displaying it even if nid3 is NULL.

Good idea. In fact, can't we replace almost all uses of id3->path in there
with a call to playlist_peek(0) for the current song and playlist_peek(1) for
the next one? Do we even need the path in the mp3entry struct?

> BTW. I think there might be a bug in the patch: you're not checking whether
> or not id3 is NULL before trying to access it in get_tag().

Oops. You're right!

 Daniel Stenberg -- --
Received on 2004-01-16

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