From: Hardeep Sidhu <hardeeps_at_pobox.com>
Date: Fri, 16 Jan 2004 00:50:31 -0800

Daniel Stenberg wrote:

>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?
>
>

playlist_peek() needs to read the track name from disk if it's stored in
the playlist or control file so we probably want to avoid that whenever
possible. We can certainly cache this information in a different
location if wanted but I don't see anything wrong with leaving it in the
mp3entry struct.

-Hardeep
Received on 2004-01-16