I don't like how this silently aborts if the start is out of bounds. It should do the same clamping as is done for the end.

this is already assured

Again, these should probably be s16 bounds.

Again, these should probably be s16 bounds.

Why not start directly at the s16 boundary ±1? 40k is a bit far away.

IMO the warning here is not needed. More importantly, should this really abort the entire raycast? A raycast could start outside the map but still hit something valid. It might be better to clip the coordinates to the first point of the ray that is inside the map ([like this I guess?](https://webcache.googleusercontent.com/search?q=cache:UA4_BJV5AzcJ:https://www.scratchapixel.com/lessons/3d-basic-rendering/minimal-ray-tracer-rendering-simple-shapes/ray-box-intersection))