anvilvapre

Looking at the code for the first time it seems that the `InfiltrateFor` classes share a lot of the same code. And that this pull request even add more duplicate...

That would only require the size of an array to be updated to i.e. 8.

Rebase requires me to check each line of the other pull request. As often with old open pull requests. Feel free to take this over. Just didn't make sense to...

Side note. This seems also like general functionality you would like to have in one place rather than all traits implementing it over time? #19582

Please validate that you see a similar improvement. After consider merging this. Or create your own implementation. Can easily be swapped with a different implemetation in future.

> "The key is to pass the IComparer as a generic arg to the priority queue, and to make the comparer a struct. This allows the JIT to devirtualize the...

also back then i believe i measured total complete path planning time, not time in priority queue. this was before hpf. so it is a big part of the total...

> please create a pull request for your version, so we can close this one? not sure if you can commit here as well. whatever suits.

would be nice if the harvester also considers the value, i.e. will move to gems a bit further away - over nearer by ore.

Didn't rebase in the past on purpose as it feels too much like a hack to work around the contineous unnessarry updates. Let me close it.