search

CyclopsMC / CapabilityProxy

Access block capabilities from multiple sides
MIT License
5 stars 5 forks source link

Fix Stack Overflow in ranged proxy #21

Closed LexManos closed 5 years ago

LexManos commented 5 years ago

Using a Block level field is just asking for concurrency issues. So pass the stack in via the Raytrace value.

Also, fixed NPE that you cause in other mods by returning null from getCapability/getTarget.

rubensworks commented 5 years ago

Thanks! I like your solution to pass the chain in as a BlockRayTraceResult, I haven't thought of that.