"Add To Playlist" patch

[kenshin]

It could be a problem with the codec loader, which has been fixed, but perhaps still exists in lennys build which is a few days old.

The probelm occurs whenever the codec changes, so if your first 4 tracks are MP3, they'll be fine, then the 5th is OGG, the WPS will display OK, but the playback remains at 0.00

Ah. Thanks. Very likely. I have quite the mix of MP3, Ogg, and FLAC files in that playlist.

[lenny]

I updated my build it you want to try that. :p

[kenshin]

Thanks for the offers. I just had to refresh my local CVS and build it again. I've had a local CVS tree for about 6 months but really haven't been making custom builds lately.

[dobz]

I think i found a problem with this patch

If you rename a playlist then go to add songs to it again it says "the playlist no longer exists" but of course it does because you are playing it.

also you have to add .m3u to the playlist name which maybe it should automatically do itself?

EDIT: it looks like under some circumstances the contents of ".playlist_catalog" gets erased

[Assimalyst]

I'm having trouble patching the latest build with this patch. Get a number of HUNK failures?? Anyone know what the problem might be?

Thanks

*sigh* Wish this was in CVS

[dunno]

If I remember correctly, Linus is waiting for shebb to do something to his patch and then he'll committ it. I agree that this is a good idea and would be great if it was implemented, *sigh*, well more lobbying is required on this one so that it doesn't fade into obscurity.

[Assimalyst]

I believe you're right.

Haven't seen much of shebb in the forums lately, just checked shebb's last active though, and it was this morning. So shebb if you're out there, it would be very cool if you could add the finishing touches!!

*sigh* wish I could help, might have to investigate learning C.

[dunno]

err where is shebbs patch on the tracker, I can't find it, do you see it ?.

[Assimalyst]

don't think it's on the tracker, at least not the lastest version. Linus edited one here: http://sourceforge.net/tracker/index.php?func=detail&atid=439120&group_id=44306&aid=1255322

But shebb has since improved it check these thred posts:
http://forums.rockbox.org/index.php?topic=2359.msg16592#msg16592
http://forums.rockbox.org/index.php?topic=2359.msg16593#msg16593
http://forums.rockbox.org/index.php?topic=2359.msg16598#msg16598

I tried to add them to sourceforge but couldn't get access to the existing patch in the first link.

[LinusN]

Also, the latest patch in this thread is not ready for CVS...

[dunno]

I really need to phylosophize the playlist functionality. I feel that the Rockbox playlist feature could really *kick ass* with the ability to create additional playlists on the fly and be able to add files to any playlist at will. I know and am aware that there is a patch available which does this, but the original developer of the patch appears to have lost interest, so could someone else *pick up the ball*?, I fear that this wonderfull enhancement to Rockbox will die a quiet death, unless ofcourse I start learning how to write zeros and ones myself, in which case expect the feature in 3 years.

Please add your support if you think that Multiple On The Fly Playlists (MOTFP  :-)  )  would add value to Rockbox and enhance user experience.

pretty please, Linus, preglow, amiconn, Cassandra, Zagor, Badger, Lostlogic, Slasheri, Linuxstb, safetydan, and all others who have the ability to write zero's and one's.

[LinusN]

I would pick up the ball if I had the time...

[dunno]

Thanks Linus.

Maybe safetydan has picked up the ball....

[Assimalyst]

Please add your support if you think that Multiple On The Fly Playlists (MOTFP  :-)  )  would add value to Rockbox and enhance user experience.

Two hands up!! would love to see this in CVS.

Maybe safetydan has picked up the ball....

Where have you heard this wonderful little rumour? i'm getting excited!!!!

[Shebb]

As of now, I have updated everything Linus suggested except changing it to use gui_synclist directly.  This I will have to get round to using because something external to my code seems to have messed things up for me.  When the context menu is used within the playlist menu and something like 'remove' is selected, that action will occur, but the click will not get removed from what ever queue it is in so the top playlist is then selected and played. 

I have just gotten back to college which means work and partying has cut out the time I had for something that has nothing to do with my studies.  This means I have not looked through the gui_synclist code much.  Whether I could code something cleanly with it after I get round to looking at it is also a question so I would appriciate if someone could step in here and add all this functionality.  I could repost the newly modified patch with the parts that require changing highly commented if someone would be interested in finishing it.  Otherwise I will give it a stab if I have some real time.