Rockbox

Tasklist

FS#11263 - Radio art

Attached to Project: Rockbox
Opened by Jonathan Gordon (jdgordon) - Wednesday, 12 May 2010, 15:15 GMT
Last edited by Jonathan Gordon (jdgordon) - Sunday, 16 May 2010, 11:14 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This makes the %Cl and %C tags work in the fm screen skin!

to use it you must be in presets mode, and have a preset file loaded. then in the fmpresets folder you need one .bmp for each station named the same as it is in the presets file.

this first version only loads bmp...
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Sunday, 16 May 2010, 11:14 GMT
Reason for closing:  Accepted
Additional comments about closing:  in r26078
Comment by Michael Chicoine (mc2739) - Thursday, 13 May 2010, 02:42 GMT
Updated patch to correct compile errors (changed strncpy to strlcpy)
Comment by Jonathan Gordon (jdgordon) - Thursday, 13 May 2010, 11:00 GMT
this version stores 10 images so there should be no load at all when changing stations if the image was loaded already.
It will fail if more than 10 images need to be loaded though (simple fix but running out now)
Comment by Jonathan Gordon (jdgordon) - Thursday, 13 May 2010, 14:52 GMT
Fix %?C not working and %C for radio not working in sbs.

dump the oldest image if we need to load more than 10, or there isnt enough room on the buffer (pretty much untested).

This is commitable now except going into the recording screen then back into fm with a station expecting an image will very likely crash the player!

Also seach for .jpg in the same folder (.bmp first though.. order can be changed)
Comment by Michael Chicoine (mc2739) - Thursday, 13 May 2010, 17:47 GMT
%?C and .jpg support tested - no problems detected

Entering recording screen from fm screen caused: Data abort at 00008ea4 (0)
This happens even after stopping the fm radio. If you start playback and then go to the recording screen, it does not crash.
Comment by Jonathan Gordon (jdgordon) - Friday, 14 May 2010, 02:26 GMT
thanks, yeah that is what I was expecting. I'm not entirely sure how to make this perfectly safe but I'll play today if I get a chance
Comment by Jonathan Gordon (jdgordon) - Friday, 14 May 2010, 06:11 GMT
This should fix the recording problems. It isnt really as clean as I would like though... please let me know if it does work (radioart should stop working as soon as you try recording. stopping the recording should make it work again)
Comment by Michael Chicoine (mc2739) - Saturday, 15 May 2010, 00:24 GMT
No more crashes with this latest patch. On e200, when recording, it goes to the recording screen, so no radioart anyway.

Loading...