FS#7729 - adds delete option to menu for jpeg viewer

Attached to Project: Rockbox
Opened by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 03:49 GMT
Last edited by Steve Bavin (pondlife) - Wednesday, 10 October 2007, 10:47 GMT
Task Type Patches
Category Plugins
Status Unconfirmed
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 0%
Votes 3
Private No


Delete current file from within the jpeg viewer plugin.
Adds delete to the options in the menu
Also highlights the last file viewed in the file browser on quit.
This task depends upon

Comment by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 15:35 GMT
Plugin now exits after delete of last file.
Comment by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 15:36 GMT
Here is the patch
Comment by Nils Wallménius (nls) - Saturday, 08 September 2007, 16:14 GMT
That last patch includes lots of unrelated changes
Comment by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 17:35 GMT
Sorry about that here is jpeg_delete1.1.diff
only jpeg stuff.
Comment by Nils Wallménius (nls) - Sunday, 09 September 2007, 13:00 GMT
I tried the patch and I do not like the two splashes that follow each other when you try to delete a file, I think you should do something like the delete_handler() function in onplay.c maybe just adding that function to the plugin api would be easiest and would also make the ui more consistent.

When deleting a file and try to jump to the next I sometimes got "err opening /G: -1" and sometimes right after deleting "unsupported" splashed for files which are supported and view just fine.

This was all on the simulator for iriver h300.
Comment by Gerritt Gonzales (GRaTT) - Sunday, 09 September 2007, 22:21 GMT
Fixed the delete menu, no more splashes.
Error handling fixed, does no try to open deleted file.
Tested on target sansa and in sim.
Comment by Victor (BukTop) - Sunday, 07 October 2007, 15:41 GMT
Good one. Works fine. I like the feature.
Comment by Gerritt Gonzales (GRaTT) - Wednesday, 10 October 2007, 05:36 GMT
Some small changes to the delete screen
to make it more like the delete_handler
I could not successfully use delete_handler
function in plugin.c/h. Trouble with static.
Comment by Corwyn Simpson (siber_cat) - Wednesday, 21 January 2009, 19:13 GMT
Any chance someone wants to adapt this patch to the split up jpeg plugin?
Comment by Gerritt Gonzales (GRaTT) - Friday, 20 March 2009, 05:26 GMT
Updated for the split up jpeg plugin.
Two patches
jpeg_delete1.4.diff just the delete option,
jpeg-rotate-delete1.0.diff includes fs#9497 the rotate option.
Comment by Gerritt Gonzales (GRaTT) - Saturday, 20 June 2009, 05:27 GMT
updated for June 19 2009 Version: r21362M-090620
menu changes.
Comment by Gman (Thecoolgman) - Saturday, 20 June 2009, 06:25 GMT
Thank you.