Describe the bug
A list of code locations where expressions get evaluated more than once.
Expected behavior
Expressions should only ever be evaluated once per get()/execute() call (excepting intentional circumstances like mapping expressions or similar)
Locations
|
AnvilGuiWrapper anvilGuiWrapper = anvilGuiWrapperExpression.getSingle(event); |
|
entityExpression != null ? entityExpression.getSingle(event) : null, |
|
|| igniteCauseLiterals.getAll().length == 0 |
(not a big deal because literals only, but a bit wasteful)
|
return test(boundingBox1, event, box1 -> test(boundingBox2, event, box1::overlaps), isNegated()); |
|
test(objects, event, object -> { // todo: utils |
|
return worldExpression.stream(event).flatMap(world -> boundingBoxExpression.stream(event).flatMap(boundingBox -> |
|
return worldExpression.stream(event).flatMap(world -> world.getNearbyEntities(boundingBox, filter).stream()); |
(tracked in
#63)
|
&& test(equipmentSlotExpression, event, |
|
bodyPartExpression.stream(event) |
|
.flatMap(armorStand -> bodyPartExpression.stream(event) |
|
.flatMap(entity -> equipmentSlotExpression.stream(event) |
|
.forEach(entity -> equipmentSlotExpression.stream(event) |
|
return test(offlinePlayerExpression, event, offlinePlayer -> { |
|
rotationExpression.stream(event), |
|
for (EntitySnapshot snapshot : snapshots.getAll(event)) { |
|
.flatMap(axis -> getExpr().stream(e) |
