<table><tr><td style="">davidhurka added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10455">View Revision</a></tr></table><br /><div><div><p>I read the reported test results as if this doesn’t work correctly. However, without this patch it doesn’t work at all, so that may already be an argument to accept this patch.</p>

<p>Text handling and search is IMHO broken in general, and needs to be redone. The current approach reorders all TextEntities, to give a nice left-to-right-top-to-bottom order. That is nice in general, but I think it should be done only on paragraph level, not on character level.</p>

<p>What needs to happen next for this patch, is probably to rebase it on master, and submit it on invent.kde.org. Then we can see together how to fix the mentioned memory leaks.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10455">https://phabricator.kde.org/D10455</a></div></div><br /><div><strong>To: </strong>fahadalsaidi, Okular, aacid, ltoscano<br /><strong>Cc: </strong>davidhurka, yaron, okular-devel, chfanzil, ngraham, johnzh, andisa, siddharthmanthan, maguirre, fbampaloukas, joaonetto, kezik, darcyshen, aacid<br /></div>