Fixed up the commitRequired checks.

This commit is contained in:
Matthew Miller 2018-12-09 15:15:34 +10:00
parent 5f2c77b719
commit 6f3016c7f0
4 changed files with 8 additions and 6 deletions

View File

@ -386,7 +386,9 @@ public class EditSession implements Extent, AutoCloseable {
* Returns queue status. * Returns queue status.
* *
* @return whether the queue is enabled * @return whether the queue is enabled
* @deprecated Use {@link EditSession#getReorderMode()} with MULTI_STAGE instead.
*/ */
@Deprecated
public boolean isQueueEnabled() { public boolean isQueueEnabled() {
return reorderMode == ReorderMode.MULTI_STAGE && reorderExtent.isEnabled(); return reorderMode == ReorderMode.MULTI_STAGE && reorderExtent.isEnabled();
} }

View File

@ -73,7 +73,7 @@ public class ChunkBatchingExtent extends AbstractDelegateExtent {
} }
public boolean commitRequired() { public boolean commitRequired() {
return enabled && batches.size() > 0; return enabled;
} }
@Override @Override
@ -88,7 +88,7 @@ public class ChunkBatchingExtent extends AbstractDelegateExtent {
@Override @Override
protected Operation commitBefore() { protected Operation commitBefore() {
if (!enabled) { if (!commitRequired()) {
return null; return null;
} }
return new Operation() { return new Operation() {

View File

@ -199,7 +199,7 @@ public class MultiStageReorder extends AbstractDelegateExtent implements Reorder
} }
public boolean commitRequired() { public boolean commitRequired() {
return enabled && stages.values().stream().anyMatch(stage -> stage.size() > 0); return enabled;
} }
/** /**
@ -248,7 +248,7 @@ public class MultiStageReorder extends AbstractDelegateExtent implements Reorder
@Override @Override
public Operation commitBefore() { public Operation commitBefore() {
if (!enabled) { if (!commitRequired()) {
return null; return null;
} }
List<Operation> operations = new ArrayList<>(); List<Operation> operations = new ArrayList<>();

View File

@ -114,12 +114,12 @@ public class FastModeExtent extends AbstractDelegateExtent {
} }
public boolean commitRequired() { public boolean commitRequired() {
return (enabled && !dirtyChunks.isEmpty()) || (postEditSimulation && !positions.isEmpty()); return enabled || postEditSimulation;
} }
@Override @Override
protected Operation commitBefore() { protected Operation commitBefore() {
if (!enabled && !postEditSimulation) { if (!commitRequired()) {
return null; return null;
} }
return new Operation() { return new Operation() {