D15301: Use std::unique_ptr for two private data members

[Oliver Sander]

sander added inline comments.

INLINE COMMENTS

> tobiasdeiminger wrote in pageviewannotator.cpp:515
> This assumes `m_pageView->textSelectionForItem` transfers ownership for the returned `RegularAreaRect*`. That's actually true, but we can only tell it by looking at the code inside `TextPage::textArea`.
> 
> Would be nice if we could make ownership explicit throughout the call chain. But I'm afraid it's problematic because `TextPage::textArea` is part of public interface and we don't want to change it?
> 
> If you can't change it, just go on. It's not a problem introduced by your patch but already existed before.

> Would be nice if we could make ownership explicit throughout the call chain.

Agreed

> But I'm afraid it's problematic because TextPage::textArea is part of public interface and we don't want to change it?

Not sure. But I'd rather leave that to a separate diff.

REPOSITORY
  R223 Okular

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

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