Alexander Harkness
Alexander Harkness
Oh yes, this is a BlockHandler not a BlockEntity! This won't work at all with more than one piston in the world, it needs something to be stored per-piston.
Possibly in the redstone simulator? Not sure though, this needs a bit of thought.
We don't have a block entity for pistons at the moment (at least I think so). Creating one and using it to store this value makes sense.
Time for a review then, I'll try rebasing and have a go this evening.
Yes, we have a darwin slave provided by G.Reaper on the Discord.
[email protected] is the generic email address we can use for all things like that.
I will need to look more into the change of map -> unordered map, but how about just getting the chunk once and using UnboundedRelGetBlock, which should be much faster...
> And if the block isn't in the same chunk we ignore it. UnboundedRel functions automatically chase the chunk pointers through neighbours, so they'll get the block quickly, even if...
This is still a draft right, until the other chunkmap-elimination changes have been made, and a better analysis of the performance impacts carried out?
Please do, since there is nobody already working on this you can just submit a PR when you're done :)