hey all,
Well, I had a little play with the playlist catalog feature and I like
it, but one thing I found which should maybe be changed is if you go
playlist catalog > add to new playlist and type the name of an
existing .m3u it should ask if you want to overwrite it instead of
rejecting it.
Also it should create the /Playlists folder instead of complaining.
But thats not why im writing this.
I noticed the changes made to playlist.c and in playlist_catalog.c
start to use a sort of directory walker which I implemented during the
freeze. So what I have done is fixed up my patch to go with latest cvs
and made the playlist catalog use it instead of its own walker
function.
the url is http://www.rockbox.org/tracker/task/5623
This patch, by the way, replaces the 3 (now 5) occurances of pretty
much the same code in playlist.c and playlist_catalog.c, and it is
also iterative instead of recursive, which in theory should be
slightly faster?
Received on 2006-07-19