




tune_zone for example.
https://github.com/ddnet/ddnet/assets/13364635/43fb42a6-2e6b-409b-84dc-f92c0fd9b9a7


















































GameServer()->GetPlayerChar(ClientID)
but i want something like GameServer()->GetPlayer(ClientID)



GameServer()->GetPlayerChar(ClientID)->GetPlayer()




GameServer()->GetPlayerChar(ClientID)->GetPlayer() 











TextColor calls are unnecessary. Now the text color is only set and reset once when rendering each entities layer.







Text call is simply too inefficient

CAutoMapper only needs CLayerTiles which has been moved to layer_tiles.h.



IsInside check.































2













bind x +toggle cl_dummy_fire 1 0 wasn't very snappy - as in, simply wouldn't process the button press, but only sometimes.
I then wondered why this was seemingly fixed when cl_dummy_copy_moves was set to 1, even if all the code for it was empty.
Like the comment says, it's supposedly an ugly workaround, so if you don't want to endorse this kind of change then I guess don't add it, but it probably doesn't break anything anyw...