Rockbox

Please somebody help me with the makefile. There are three source .c files in plugins/shortcuts: shortcuts_common.c, shortcuts_view.c, and shortcuts_append.c (and one .h file)

Out of these three files, two .rocks should be produced:
1. shortcuts_common.c + shortcuts_view.c → shortcuts_view.rock
2. shortcuts_common.c + shortcuts_append.c → shortcuts_append.rock

How can this be done?

I somehow managed to produce a patch file with duplicated patches. Attached is the corrected patch.

   shortcuts2.patch (49.8 KiB)

Now everything gets compiled and linked (many thanks to linuxstb). Viewer works. But the appender doesn’t get loaded. On the sim, I get error message “Incompatible version”. And have no idea about the reason. Linking problem?

   shortcuts3.patch (30.5 KiB)

Now everything works. Thanks go to linuxstb again for spotting the problem in shortcuts_append.c. This could be committed if you ask me. Feedback is welcome.

   shortcuts4.patch (48.6 KiB)

I can understand most of your changes, except splitting the plugin in two… It wasn’t that big and complicated, was it?

Now everything works as in the original, i.e. selecting a file entry places the cursor at the file in the file browser. Selecting a dir entry enters that dir and places the cursor at the first file inside it.

Since I was one of the initiators for the creation of the plugin and am one of its very few users (if not the only one could you please commit it asap.

@petur: after the splitting, each part has its own little but well defined task. Before splitting, if you selected a .link file and wanted to add it to the shortcuts, it was shown instead → not the principle of the least surprise IMHO. Another thing was that the association of the plugin with .link files was hard coded in the plugin. IMHO it should be done in viewers.config only.

   shortcuts5.patch (48.5 KiB)

Corrected patch. Now we correctly determine if it’s a file or a dir (sim and target behave differently when calling open()).

   shortcuts5-2.patch (48.6 KiB)

some remarks:
1) the opening braces of a function should be on a new line (this is not yet in CONTRIBUTING but it will be soon)
2) I do not understand how the shortcuts to shortcuts works. With your patch you can add it but afterwards it doesn’t look like you can do much with it (I thought it would open the shortcut file)

to 1) I’ll correct this (are there tools to do that automatically?)

to 2) Shortcuts are just a way to quickly go to a certaint dir/file. It doesn’t dictate what you should do with it then, just takes you there in the file browser. Just a shortcut. It doesn’t “play” the file. So you could put a reference to a link file into another link file and quickly jump from dir to dir

there were not that many {} violations, I did it here in order to test and it took < 5m ;)

thanks for clarifying 2)

Some further modifications:

1. Corrected braces
2. Removed malloc, work with an array instead
3. After deleting an entry, stay in the list
4. If an item with no more existing path/file is selected, display a splash but stay in the list
5. Properly handle connecting to USB

   shortcuts6.patch (48.7 KiB)

Slightly better behaviour after deleting an entry: if it was the last entry, exit the plugin.

Ah, wrong button clicked! Now with the patch.

   shortcuts6-2.patch (48.7 KiB)