Rockbox

Tasklist

FS#7437 - New menu item to delete the current playing file. Tested on SANSA, should work for all players.

Attached to Project: Rockbox
Opened by John Zhou (timevision) - Sunday, 15 July 2007, 03:02 GMT
Last edited by Jonathan Gordon (jdgordon) - Tuesday, 31 July 2007, 12:26 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 3
Private No

Details

Hey guys, and gals who are brave enough,

Here is a very easy patch that allows you to delete the current playing music file via the WPS menu (only changed one line of code in onplay.c).

I tested it on my Sansa e280 player only, but it should work on all players as it's just one line of code change on the latest build.

I'm aware that there is already a similar patch (ID 4812) made a while ago, but it didn't work on the latest Rockbox build. The code was completely outdated.


------ Reason for this Patch ------

I started using Rockbox for my Sansa e280 player a week ago, hoping it could allow me total control of all the files stored on the player just like a computer.

Everything is just perfect, except to my disappointment Rockbox still doesn't allow me to delete the currently playing music files via the WPS menu.

I just need this feature in order to organize my musics more efficiently because I listen to a lot of new music. This will allow me to just upload all the new music files onto the player, listen to them, and easily delete the unwanted ones.

I'm now 100% happy with Rockbox! Thanks to the team for making this wonderful Open Source firmware so that even newbies like me can contribute.


------ Known Issues ------

There is just one small issue with this patch, but I consider it a feature, not really a bug.

After the current playing file is deleted it will continue to play, probably from the memory buffer. You can allow it to finish or you can just press NEXT to skip it. I'm not going to bother adding more code to automatically skip to the next song because I'd like to give the poor song a last chance to be at least played fully once before it is wiped forever : D



BTW, for newbies:

1. In case you don't know, you must compile your own Rockbox build in order to use this patch, and all other patches.
See details at : http://www.rockbox.org/twiki/bin/view/Main/SimpleGuideToCompiling

2. You must then
apply the patch before compiling the build again.
See details at :
http://www.rockbox.org/twiki/bin/view/Main/SimpleGuideToCompiling#Adding_Patches


Enjoy!
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Tuesday, 31 July 2007, 12:26 GMT
Reason for closing:  Accepted
Comment by Dominik Riebeling (bluebrother) - Sunday, 15 July 2007, 21:59 GMT
This will definitely trigger a known bug -- see the related task  FS#5642 .

Loading...