> On Sept. 25, 2013, 9 p.m., Albert Astals Cid wrote: > > core/textpage.cpp, line 120 > > <http://git.reviewboard.kde.org/r/112135/diff/2/?file=185753#file185753line120> > > > > This is an unrelated fix, right?
Yes, I noticed it by reviewing the code. I have also added a test for it (testHyphenWithYOverlap). - Jaan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112135/#review40819 ----------------------------------------------------------- On Oct. 5, 2013, 8:20 p.m., Jaan Vajakas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112135/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2013, 8:20 p.m.) > > > Review request for Okular. > > > Bugs: 323262 and 323263 > http://bugs.kde.org/show_bug.cgi?id=323262 > http://bugs.kde.org/show_bug.cgi?id=323263 > > > Repository: okular > > > Description > ------- > > This patch solves Bug 323262 and Bug 323263. I also refactored and simplified > the code a little. By removing unnecessary calls to toLower in > TextPagePrivate::findTextInternalForward and > TextPagePrivate::findTextInternalBackward I also fixed a small bug: the > letter capital I with dot above (U+0130) did not match itself in > case-insensitive mode on Qt 4.8.4 (U+0130 still does not match lowercase i > (U+0069), which can be considered another bug, that I didn't fix (although > this behavior conforms to the Unicode case folding rules)). > > (I did not implement the Knuth-Morris-Pratt algorithm that I promised in a > comment of Bug 323263 because on second thought I find that the win, if any, > would probably be negligible except for some very special documents and > special query strings.) > > > Diffs > ----- > > core/textpage_p.h 8ecf0c9 > core/textpage.cpp 855942d > tests/searchtest.cpp 495107d > > Diff: http://git.reviewboard.kde.org/r/112135/diff/ > > > Testing > ------- > > > Thanks, > > Jaan Vajakas > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel