-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/113986/
-----------------------------------------------------------

(Updated Feb. 17, 2014, 6:29 a.m.)


Review request for Okular.


Changes
-------

further adjustments


Repository: okular


Description
-------

First shot for multiple tiles managers way. Let's face it -- it's nasty. But 
the compiled okular is displaying my testing pdf without crashing, so at least 
that.

It's straighforward implementation. Every single place, where there were call 
for (or with) TilesManager, now has a DocumentObserver as companion. The 
m_tiledManager reference in PagePrivate was changed to QMap<DocumentObserver, 
TilesManager>.

It's adding more code then I expected, but I think that now, when every request 
can be back-supported by TilesManager, all the request can now be served 
trought them and not by that direct Pixmap cache in Document. So with that, it 
would actually be more deletion (imho).


Diffs (updated)
-----

  active/components/documentitem.cpp aaf98b3 
  core/document.h fe296e0 
  core/document.cpp 19eaa70 
  core/document_p.h ef25169 
  core/generator.cpp 23b274b 
  core/generator_p.h 9021763 
  core/page.h bc8c09e 
  core/page.cpp e194523 
  core/page_p.h 63d4da1 
  part.cpp 283e91a 
  ui/pagepainter.cpp d5d9c3e 
  ui/pageview.cpp 65967bf 

Diff: https://git.reviewboard.kde.org/r/113986/diff/


Testing
-------


Thanks,

Michal Humpula

_______________________________________________
Okular-devel mailing list
Okular-devel@kde.org
https://mail.kde.org/mailman/listinfo/okular-devel

Reply via email to