From: Hardeep Sidhu <hardeeps_at_pobox.com>
Date: Thu, 15 Jan 2004 09:50:17 -0800

Hi Daniel,

Daniel Stenberg wrote:

>I just submitted a patch for displaying info on the WPS about the next song. I
>would guess that this could make a good combination with the alternating lines
>stuff!
>
>The patch is available from these places:
>
>http://daniel.haxx.se/nextsong.patch
>http://sourceforge.net/tracker/index.php?func=detail&aid=877636&group_id=44306&atid=439120
>
>I made this a patch to first get comments and hear what people say, and then
>we can decide if this is a good approach/idea or not.
>
>

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.

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().

-Hardeep
Received on 2004-01-15