Skip to content

Commit 9a552d1

Browse files
authored
Merge pull request #1499 from NASA-AMMOS/bugfix/queue-remove-effect
Fix DiscreteEffects.remove
2 parents d6fecb1 + 293797b commit 9a552d1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

contrib/src/main/java/gov/nasa/jpl/aerie/contrib/streamline/modeling/discrete/DiscreteEffects.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -115,10 +115,10 @@ public static <T> Optional<T> remove(MutableResource<Discrete<List<T>>> resource
115115
final var currentQueue = currentValue(resource);
116116
if (currentQueue.isEmpty()) return Optional.empty();
117117

118-
final T result = currentQueue.get(currentQueue.size() - 1);
118+
final T result = currentQueue.get(0);
119119
resource.emit(name(effect(q -> {
120120
var q$ = new LinkedList<>(q);
121-
T purportedResult = q$.removeLast();
121+
T purportedResult = q$.removeFirst();
122122
if (!result.equals(purportedResult)) {
123123
throw new IllegalStateException("Detected effect conflicting with queue remove operation");
124124
}

0 commit comments

Comments
 (0)