MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-09-27 10:03:11 +07:00
Code Issues Packages Projects Releases Wiki Activity

diff: code review

Browse Source 
IdRunnable is unnecessary - additional callbacks will do nothing, because myQueuedApplyRequest will be cleared
This commit is contained in:
Aleksey Pivovarov
2015-04-13 19:50:56 +03:00
parent 249f08fec8
commit 202f6d8cef
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
platform/diff-impl/src/com/intellij/diff/impl/DiffRequestProcessor.java
+7 -6
View File
@@ -240,19 +240,20 @@ public abstract class DiffRequestProcessor implements Disposable {
// Abstract
//
@Nullable private ApplyData myApplyData;
@Nullable private ApplyData myQueuedApplyRequest;
@CalledInAwt
protected void applyRequest(@NotNull DiffRequest request, boolean force, @Nullable ScrollToPolicy scrollToChangePolicy) {
myIterationState = IterationState.NONE;
myApplyData = new ApplyData(request, force || (myApplyData != null && myApplyData.force), scrollToChangePolicy);
IdRunnable task = new IdRunnable(this) {
force = force || (myQueuedApplyRequest != null && myQueuedApplyRequest.force);
myQueuedApplyRequest = new ApplyData(request, force, scrollToChangePolicy);
Runnable task = new Runnable() {
@Override
public void run() {
if (myApplyData == null || myDisposed) return;
doApplyRequest(myApplyData.request, myApplyData.force, myApplyData.scrollToChangePolicy);
myApplyData = null;
if (myQueuedApplyRequest == null || myDisposed) return;
doApplyRequest(myQueuedApplyRequest.request, myQueuedApplyRequest.force, myQueuedApplyRequest.scrollToChangePolicy);
myQueuedApplyRequest = null;
}
};