icybear/fabric-carpet
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

removed potential ConcurrentModificationException chance, part one for 304

Browse Source
This commit is contained in:
gnembon committed 2020-06-01 21:10:42 -03:00
1 parent 59be8b3fb4
commit 01a2d9ea6e
1 file changed
+7 -1
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/carpet/script/CarpetEventServer.java
+7 -1
View File
@@ -126,7 +126,13 @@ public class CarpetEventServer
List<LazyValue> argv = argumentSupplier.get(); // empty for onTickDone List<LazyValue> argv = argumentSupplier.get(); // empty for onTickDone
ServerCommandSource source = cmdSourceSupplier.get(); ServerCommandSource source = cmdSourceSupplier.get();
assert argv.size() == reqArgs; assert argv.size() == reqArgs;
callList.removeIf(call -> !CarpetServer.scriptServer.runas(source, call.host, call.function, argv)); // this actually does the calls List<Callback> fails = new ArrayList<>();
for (Callback call: callList)
{
if (!CarpetServer.scriptServer.runas(source, call.host, call.function, argv))
fails.add(call);
}
for (Callback call : fails) callList.remove(call);
} }
} }
public boolean addEventCall(ServerCommandSource source, String hostName, String funName, Function<ScriptHost, Boolean> verifier) public boolean addEventCall(ServerCommandSource source, String hostName, String funName, Function<ScriptHost, Boolean> verifier)