Max
Max
With both Forge and Fabric now having native datagen APIs respectively, this should probably be reconsidered soon-ish ^^
There seems to be something registering a packet with a null id... weird
I've added a null-check for packet ids, so at the very least we should be able to find out the culprit from here (it... might even be Architectury API itself...
Superseded by #302
In that case, see my comments regarding the cancelling component ^^ There *has* to be a better way to do that :/
Please resolve the merge conflicts so this can be put on last call
The way I'm reading this stacktrace, the map's size seems to increase *after* it has allocated the array for `KeySet.toArray`, which leads me to believe this is a concurrency issue...
Going to mark this as "missing api" for now, as Dan already said we **likely will not be implementing this** unless a standard emerges within Fabric API itself, as there...
So some quick notes on this, mostly originally taken from Discord: Originally, Cloth was supposed to be the de-facto cross-platform config mod, but seeing as - it's an external dependency...
Either way, bumped the priority of this to medium since we know of mods where this would be immediately useful to have now. We should probably have a short discussion...