Rewrite coordinate system

[Makkkkus]

Currently, we have two coordinate spaces in the game; one for entities, and another for tiles. Moving between these two is pretty easy and can be done with a simple bitwise operation, but that doesn't make it optimal.

We should instead make entities and tiles use the same coordinate space, for ease of use.

In the old system this would be done with:

int tile = entityPos >> 4;

While in the new system you can just do:

int tile = (int) entityPos;

It isn't such a big change on the surface other than that the latter would be much more intuitive, but this will make movement much more accurate and specific, as well as easier to work with.

[lino-levan]

Is anyone actively working on this? If not, I would like to be assigned to this

[afyber]

I am questioning whether or not this would actually be a good change to make, since the point of the original system is that: even in the entity coordinates, they are stored as an integer, and there is therefore no floating-point-error. So changing to this system could decrease performance possibly.

[afyber]

Basically I don't think it's a good idea

I'll wait for you opinion, though

[Makkkkus]

The point of this was to fix issue #172, so that we could do floting point calculations, and woudn't the performance impact be miniscule?

[afyber]

Good point.

[lino-levan]

It's been a few years, sorry about that. I don't know if this makes any sense. I don't see how #172 is related to this in any way?