I don't quite understand what you're saying...

I was wondering if I should rewrite my commit to check "getHandle().activeContainer.a(player)", which would end up calling something like this for each different inventory that was being checked:

https://github.com/Bukkit/CraftBukkit/blob/master/src/main/java/net/minecraft/server/TileEntityChest.java#L159

This would replace the distance check. Did I misunderstand your comment above?