Rockbox

Forgot to mention, since resize-on-load relies on loading an on-disk bitmap, the pictureflow empty slide image is now installed as .rockbox/rocks/demos/pictureflow_emptyslide.bmp

PREFFERED_IMG_WIDTH/_HEIGHT are not applicable for scaling dimensions. I’d just say #define RESIZE_WIDTH/_HEIGHT LCD_HEIGHT/2 like I did it  FS#8335 .
Note that you directly influence the album art search (success rate) with changing PREFERRED_IMG_WIDTH/_HEIGHT as those are used for the size string in cover.<size>.bmp too at the search. I suppose leaving those as they are, and use other symbols for resizing target size.

ALso, it doesn’t really make sense to search for covers that already have the target dimensions, does it? :)

Can you explain the LCD_WIDTH*10/32 part? I guess this should be for portrait targets, but 1/3 the screen width is pretty small (that’s just 60 on a e200, which has 220 pixel rows). Just use LCD_HEIGHT/2. It’s fine, I’ve been using that in  FS#8335  for a while now too and it looks great.
The key to see other covers is to change center margin (i.e. the space between the center and the side cover) in the settings, not reducing the imagine size. The default value is bad anyway, it’s like miles away.

My aim with the *10/32 was to hit the apparent “suggested” size of 100px on 320×240 landscape, but that seems really quite tiny. Ideally, I think that LCD_HEIGHT should be set as the maximum and default scaled size, and perhaps that the zoom % which adjusts the “distance” of covers from the camera should be replaced with an option to select the size of the largest square album art should be loaded as. It would probably not be a bad idea to replace the +/- margin with an actual pixel value, and set the default to something reasonable based on LCD_WIDTH.
As far as PREFERRED_HEIGHT and searching for covers by size, that should probably just go away. PF can’t reasonably guess what size AA the user might have, and can’t search for every possibility. Removing the size-based search would probably also get back a little bit of the binsize that resize-on-load cost.

But keep in mind that “Zoom” can zoom without recreating the cache. Resizing needs recreation of the cache. I would mind losing that option.

I’d keep zoom, and add a bool option whether to resize or not upon the next cache creation. I guess there’s people who don’t want them resized in pf, or have bigger album art where resizing would just make them too small for them.

Removed MAX_* and PREFERRED_* entirely, as well as looking for a prescaled AA. Defined a now DISPLAY_SIZE as LCD_HEIGHT/2, and reduced default margin to make other covers more visible. Regarding making zoom change the scaled size, if I were to go that route, I feel that it would be necessary to make the cache rebuild automatically in the background, and working out from the currently centered album, instead of from the head of the list. I have no idea how to do that without making a horrible mess of it, so for right now, zoom stays as-is.

Correct buffer size calculation, it was twice as large as needed and too large for plugin buffer.

   pictureflow-scaled-load-20081... (10.9 KiB)