Add extra checks to make sure we don't accidentally suffocate player

[zford]

If blocks directly above the mine are not air, we could potentially suffocate the player. This checks the two top blocks to make sure they are air, and if they're not, TPs them to the highest spot (surface?) on the X, Z coordinate.

This additionally fixes an edge case where standing in one of the corners may not TP you out of the mine.

[jjkoletar]

Sorry for being super late to the PR.

For the first chunk, was it necessary to switch from getY() to getBlockY(), etc?

[zford]

This additionally fixes an edge case where standing in one of the corners may not TP you out of the mine.

You have to remember a block fills a space, and isn't a single point. You need to check if the player is within the space between the block edges, which expands beyond the boundries set by min[XYZ] max[XYZ]

While a block's cordinates might be 50,50,50, it fills the space between 50,50,50 and 51,51,51. So if you're standing on a block in the mine with coordinates of max[XYZ], you'll you need to check an additional block out.

You could probably use getX() and expand the max[XYZ] borders by one, but then you're doing a floating point check instead of an integer check.. in addition to an addition operation. Casting to int is probably faster, and in this case is less ambiguous, since all i really care about is the block location that the player is standing on, not the space between the blocks he is standing on.