D14752: Fix links being "lost" on save

[Albert Astals Cid]

aacid added inline comments.

INLINE COMMENTS

> tobiasdeiminger wrote in generator_pdf.cpp:653
> Guess this is the same sanity check as in document.cpp, Document::swapBackingFile, probably meaning "is it still the same document?". Could there be situations where old page count != new page count? Or where page count is equal, but content is completely different?

Yes, sanity check in case stuff did blow up for some reasons.

> tobiasdeiminger wrote in generator_pdf.cpp:655-661
> This adds a few lines of redundant code and minor special handling (wrt PDFGenerator::image). But I shouldn't complain, don't know how to do it better.

Yes, it's 3 duplicated lines, but i honestly couldn't find a good name for the new function, so i decided to just copy them.

REPOSITORY
  R223 Okular

REVISION DETAIL
  https://phabricator.kde.org/D14752

To: aacid
Cc: tobiasdeiminger, sander, okular-devel, ngraham, aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20180813/c0173bc3/attachment.html>