Rockbox mail archiveSubject: Re: Queue function
Re: Queue function
From: Jack Peel <jhp_at_cypress.com>
Date: Tue, 07 Jan 2003 11:58:10 -0800
Yes it turns out that I had a hacked cvs checkout (checked out the
directories individually instead of the entire "rockbox" package
so the patch got a little confused. I check it out and the patch works
As far as the playlist, I'm not really sure how the playlist code works.
It seems that it would have to store the playlist in some form in memory
or a temporary file since it can non-destructivly shuffle the contents.
I guess I should look at the code before I start suggesting soutions. But
it would be a great feature for me :) It also runs into the on-the-fly
editing of playlists, so maybe I'll just wait until then :)
Hardeep Sidhu wrote:
>>Seems to work fine on my archos, but that was just an inital try, I will
>>try to beat it up more later.
>>One thing that would be nice is that if you are playing a playlist, for it
>>to check and see if the song that you queued up is in the playlist and
>>temporarily remove it. That way if I want to play a song that is on
>>the playlist "now" I don't have to worry about it showing up later.
>>(I'm using this for break music during a show, and I get requests
>>from time to time). I don't know how the "current" playlist code
>>organizes the data so this may not be easy, but if it is it would be
>An interesting idea, I can see how it could be useful. However, deleting
>items from a playlist is non-trivial and potentially expensive because we
>would have to scan the playlist file. Also, handling resume correctly would
>be a little tricky. Still, this is something i'll definitely look into
>after we've got the basic functionality committed.
>>It also appears that your patch is broken again with the new CVS,
>>or I don't know how to run patch (either is quite likley). It seems
>>to have a problem patching apps/applimits.h for MAX_QUEUED_FILES.
>>I edited it in by hand and it seems to build fine.
>I just tried it out and it worked fine ("patch -p0 -i
>queue_function_4.diff"). Did you apply it to a clean version of the latest
Received on 2003-01-07