lst: update fields under lock

* they're accessed from EDT only, so the lock is unnecesary
  but this is consistent with the rest of the class

follow-up: 42dcb2b
This commit is contained in:
Aleksey Pivovarov
2016-11-22 19:26:13 +03:00
parent 84fff35443
commit cc1c966c9e
@@ -317,16 +317,20 @@ public abstract class LineStatusTrackerBase {
@CalledInAwt
private void updateRangeHighlighters() {
myToBeInstalledRanges.removeAll(myToBeDestroyedRanges);
if (myToBeInstalledRanges.isEmpty() && myToBeDestroyedRanges.isEmpty()) return;
for (Range range : myToBeDestroyedRanges) {
disposeHighlighter(range);
synchronized (LOCK) {
myToBeInstalledRanges.removeAll(myToBeDestroyedRanges);
for (Range range : myToBeDestroyedRanges) {
disposeHighlighter(range);
}
for (Range range : myToBeInstalledRanges) {
createHighlighter(range);
}
myToBeDestroyedRanges.clear();
myToBeInstalledRanges.clear();
}
for (Range range : myToBeInstalledRanges) {
createHighlighter(range);
}
myToBeDestroyedRanges.clear();
myToBeInstalledRanges.clear();
}
private class MyApplicationListener extends ApplicationAdapter {