[Okular-devel] Review Request 118293: Make DocumentInfo not be XML based. Let users ask for only the keys they need.

Albert Astals Cid aacid at kde.org
Sat May 24 14:03:06 UTC 2014


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

Review request for Okular.


Repository: okular


Description
-------

DocumentInfo is now QMap based instead of XML based. Makes internal handling easier. The generators don't do any caching anymore since the Document was already doing caching, made no sense. Now we only ask for the keys we need (except the properties dialog) so the generators that are slow can decide to return only what was asked for. Some generators (i.e. unpluck, xps or the text based ones) will still return all the info since it's calculated on load time and it's not time consuming to return everything if we already have it.


Diffs
-----

  core/document.h 06bac8a 
  core/document.cpp 504d07c 
  core/document_p.h aabd192 
  core/generator.h 506f8a8 
  core/generator.cpp ec1d1f7 
  core/script/kjs_document.cpp c3280cd 
  core/textdocumentgenerator.h ec331d6 
  core/textdocumentgenerator.cpp d598ccc 
  generators/chm/generator_chm.h 5d97009 
  generators/chm/generator_chm.cpp 84b9dde 
  generators/djvu/generator_djvu.h 93ee4de 
  generators/djvu/generator_djvu.cpp c506ce3 
  generators/dvi/generator_dvi.h d7bf8fd 
  generators/dvi/generator_dvi.cpp 0777af7 
  generators/fax/generator_fax.h 1cb7d22 
  generators/fax/generator_fax.cpp 38873ef 
  generators/kimgio/generator_kimgio.h faebd85 
  generators/plucker/generator_plucker.h 8dac4b8 
  generators/plucker/generator_plucker.cpp 4ffdead 
  generators/poppler/generator_pdf.h 1a34bb7 
  generators/poppler/generator_pdf.cpp a363842 
  generators/spectre/generator_ghostview.h ab9c33d 
  generators/spectre/generator_ghostview.cpp 7b0146a 
  generators/tiff/generator_tiff.h 9ad1de6 
  generators/tiff/generator_tiff.cpp bc692de 
  generators/xps/generator_xps.h 4c56e8c 
  generators/xps/generator_xps.cpp 2399a96 
  part.cpp 67440bd 
  ui/presentationwidget.cpp 4aaec2a 
  ui/propertiesdialog.cpp 485c2de 

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


Testing
-------

Opened a few files, works as it did. Added some extra debug, made sure that only the info we want is asked for, and that it's properly cached.


Thanks,

Albert Astals Cid

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


More information about the Okular-devel mailing list