Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8585 - jpeg_viewer: fit lcd

Attached to Project: Rockbox
Opened by Vuong Minh Hiep (vmh) - Sunday, 10 February 2008, 19:08 GMT+1
Last edited by Peter D'Hoye (petur) - Monday, 25 February 2008, 00:09 GMT+1
Task Type Patches
Category Plugins
Status Closed
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 100%
Private No

Details

find optimal scale factor for the image to fit LCD as initial scale factor
   jpeg_fit_lcd.patch (0.6 KiB)
This task depends upon

Closed by  Peter D'Hoye (petur)
Monday, 25 February 2008, 00:09 GMT+1
Reason for closing:  Rejected
Additional comments about closing:  this is not how we want the viewer to behave, sorry...
Comment by Peter D'Hoye (petur) - Monday, 11 February 2008, 23:11 GMT+1
What does this fix? max_downscale() does what your code does, so your code will always result in selecting ds_max.

Comment by Vuong Minh Hiep (vmh) - Tuesday, 12 February 2008, 09:39 GMT+1
> max_downscale() does what your code does
No, not exactly the same. If you compare both code you'll notice max_downscale() uses 'or' and mine uses 'and'. That means in max_downscale(), both width AND hight, must be smaller than the LCD, but in my code however it's sufficient if only height OR width is smaller than the LCD.

> so your code will always result in selecting ds_max
No, not always. It depends on your LCD size and the picture size.
To test it you have to use pictures of different sizes.
Comment by Peter D'Hoye (petur) - Friday, 15 February 2008, 09:54 GMT+1
To be honest, I couldn't find a picture where your code change made a difference, unless I'm doing something wrong...
Comment by Vuong Minh Hiep (vmh) - Sunday, 24 February 2008, 21:12 GMT+1
@petur: Try with these picture. Hope you'll see a difference.
   400px-USA_Antelope-Canyon.jpg (32.4 KiB)
   400px-Palace_of_Westminster,_... (18.1 KiB)
Comment by Peter D'Hoye (petur) - Sunday, 24 February 2008, 23:56 GMT+1
ok, I see the difference now, and I think I like current behaviour better than your patch: With your patch, these pictures show up partially zoomed, where SVN shows the complete picture...

Loading...