Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

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

Attached to Project: Rockbox
Opened by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 05:49 GMT+2
Last edited by Steve Bavin (pondlife) - Wednesday, 10 October 2007, 12:47 GMT+2
Task Type Patches
Category Plugins
Status Unconfirmed
Assigned To No-one
Player type All players
Severity Low
Priority Normal
Reported Version current build
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

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.
GRaTT
   jpeg_delete.diff (3.1 KiB)
This task depends upon

Comment by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 17:35 GMT+2
Plugin now exits after delete of last file.
GRaTT
Comment by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 17:36 GMT+2
Here is the patch
   jpeg_delete1.1.diff (154.6 KiB)
Comment by Nils Wallménius (nls) - Saturday, 08 September 2007, 18:14 GMT+2
That last patch includes lots of unrelated changes
Comment by Gerritt Gonzales (GRaTT) - Saturday, 08 September 2007, 19:35 GMT+2
Sorry about that here is jpeg_delete1.1.diff
only jpeg stuff.
GRaTT
   jpeg_delete1.1.diff (3.1 KiB)
Comment by Nils Wallménius (nls) - Sunday, 09 September 2007, 15:00 GMT+2
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) - Monday, 10 September 2007, 00:21 GMT+2
Fixed the delete menu, no more splashes.
Error handling fixed, does no try to open deleted file.
Tested on target sansa and in sim.
GRaTT
   jpeg_delete1.2.diff (3.7 KiB)
Comment by Victor (BukTop) - Sunday, 07 October 2007, 17:41 GMT+2
Good one. Works fine. I like the feature.
Comment by Gerritt Gonzales (GRaTT) - Wednesday, 10 October 2007, 07:36 GMT+2
Some small changes to the delete screen
to make it more like the delete_handler
screen.
I could not successfully use delete_handler
function in plugin.c/h. Trouble with static.
GRaTT
   jpeg_delete1.3.diff (4 KiB)

Loading...