D19717: Search function fails to find phrases split over two lines
Albert Astals Cid
noreply at phabricator.kde.org
Mon Apr 22 00:54:03 BST 2019
aacid added a comment.
There's a downside to this is that now if you actually put a newline character in the search it will fail where previously it worked.
On the other hand typing an actual newline character is kind of hard (i had to copy it from a newline in kate) so maybe we can just accept that noone really knew how to do that :D
Opinions?
INLINE COMMENTS
> textpage.cpp:865
> const TinyTextEntity* curEntity = *it;
> - const QString& str = curEntity->text();
> - int len = stringLengthAdaptedWithHyphen(str, it, m_words.constEnd());
> + const QString& strNotUsed = curEntity->text();
> + int len = stringLengthAdaptedWithHyphen(strNotUsed, it, m_words.constEnd());
strNotUsed seems like a weird variable name given you actually use it the two following names.
Would something like origStr or something make more sense?
> textpage.cpp:866
> + const QString& strNotUsed = curEntity->text();
> + int len = stringLengthAdaptedWithHyphen(strNotUsed, it, m_words.constEnd());
> + const QString str = QString(strNotUsed).replace('\n', ' ');
given that you're modifying this line, make len const :)
Also are you totally sure this needs to be over strNotUsed and not str? Hmmm i guess it works both ways since str is the same length as strNotUsed
REPOSITORY
R223 Okular
REVISION DETAIL
https://phabricator.kde.org/D19717
To: joaonetto, #okular
Cc: aacid, ngraham, okular-devel, joaonetto, tfella, darcyshen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20190421/5972f715/attachment-0001.html>
More information about the Okular-devel
mailing list