yetangitu/FBReaderJ
1
0
Fork 0
mirror of https://github.com/geometer/FBReaderJ.git synced 2025-10-04 02:09:35 +02:00
Code Issues Releases Wiki Activity

refactoring (compute hilites one time per page)

Browse source
This commit is contained in:
Nikolay Pultsin 2014-09-26 09:35:12 +01:00
parent 7f71050027
commit 6ffcaa246b
1 changed files with 19 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

17
src/org/geometerplus/zlibrary/text/view/ZLTextView.java
View file

@ -508,11 +508,13 @@ public abstract class ZLTextView extends ZLTextViewBase {
previousInfo = info; previousInfo = info;
} }
final List<ZLTextHighlighting> hilites = findHilites(page);
if (!hilites.isEmpty()) {
x = getLeftMargin(); x = getLeftMargin();
y = getTopMargin(); y = getTopMargin();
index = 0; index = 0;
for (ZLTextLineInfo info : lineInfos) { for (ZLTextLineInfo info : lineInfos) {
drawHighlightings(page, info, labels[index], labels[index + 1], x, y); drawHighlightings(page, hilites, info, labels[index], labels[index + 1], x, y);
y += info.Height + info.Descent + info.VSpaceAfter; y += info.Height + info.Descent + info.VSpaceAfter;
++index; ++index;
if (index == page.Column0Height) { if (index == page.Column0Height) {
@ -520,6 +522,7 @@ public abstract class ZLTextView extends ZLTextViewBase {
x += page.getTextWidth() + getSpaceBetweenColumns(); x += page.getTextWidth() + getSpaceBetweenColumns();
} }
} }
}
x = getLeftMargin(); x = getLeftMargin();
y = getTopMargin(); y = getTopMargin();
@ -823,11 +826,7 @@ public abstract class ZLTextView extends ZLTextViewBase {
preparePaintInfo(); preparePaintInfo();
} }
private void drawHighlightings(ZLTextPage page, ZLTextLineInfo info, int from, int to, int x, int y) { private List<ZLTextHighlighting> findHilites(ZLTextPage page) {
if (from == to) {
return;
}
final LinkedList<ZLTextHighlighting> hilites = new LinkedList<ZLTextHighlighting>(); final LinkedList<ZLTextHighlighting> hilites = new LinkedList<ZLTextHighlighting>();
if (mySelection.intersects(page)) { if (mySelection.intersects(page)) {
hilites.add(mySelection); hilites.add(mySelection);
@ -839,7 +838,11 @@ public abstract class ZLTextView extends ZLTextViewBase {
} }
} }
} }
if (hilites.isEmpty()) { return hilites;
}
private void drawHighlightings(ZLTextPage page, List<ZLTextHighlighting> hilites, ZLTextLineInfo info, int from, int to, int x, int y) {
if (from == to) {
return; return;
} }