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 inefficientCAutoMapper
only needs CLayerTiles
which has been moved to layer_tiles.h
.
IsInside
check.
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...