[Okular-devel] Review Request: Add greedy preloading option

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103129/#review10003
-----------------------------------------------------------


This still does not do what you want/say it does.
Reasons:
 * There's a limit of 500 pages of text, you might want to set that to INT_MAX or something like that
 * In cleanupPixmapMemory you still unload pages sometimes

I am "fine" with that behaviour, but it seems to be not what you want, or is it?

- Albert Astals Cid


On Dec. 28, 2011, 8:32 p.m., Sebastian Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103129/
> -----------------------------------------------------------
> 
> (Updated Dec. 28, 2011, 8:32 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> After discovering that the aggressive behaviour in okular is not that aggressive at all (preloads only the next 2 pages) I've written the attached patch to add a "Greedy" option. It preloads every page into memory.
> 
> 
> Diffs
> -----
> 
>   conf/dlgperformance.cpp fd0e30b 
>   conf/dlgperformancebase.ui b022f56 
>   conf/okular.kcfg 430af7c 
>   core/document.cpp 74757f1 
>   ui/pageview.cpp b3c0982 
>   ui/presentationwidget.cpp 7c3f618 
> 
> Diff: http://git.reviewboard.kde.org/r/103129/diff/diff
> 
> 
> Testing
> -------
> 
> Tested it on my local system, I'm using the modified okular with the new greedy option checked and don't have any issues so far. The memory requirement for a 580 page (color) PDF file is about 1 GiB, the bulk of it for the pixmaps. Other (e.g. unloading) behaviour is like "Aggressive". It takes a while for the whole document to be prerendered but after that it works as expected.
> 
> 
> Thanks,
> 
> Sebastian Rose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20120122/4e1cd5a7/attachment.html>