Bouncer data mover should follow bouncers chain if possible
Closed this issue · 1 comments
To my knowledge, it is not outside the realm of possibility that a chain of bouncers to a target may exist. https://github.com/ParchmentMC/Compass/blob/dev/src/main/java/org/parchmentmc/compass/data/sanitation/impl/BouncerDataMover.java should be modified to follow any possible chain to the end method.
While theoretically a chain of bouncers may exist, in practice a bouncer is usually only inserted by the compiler between two (non-bouncher) methods.
There is some cost in refactoring BouncerDataMover
to follow bouncers to the end of the chain which I'm not willing to invest in for now, since it will likely never activate. In the future, should a chain of bouncers indeed be observed in the real world, I'll re-open this issue and implement it.