refactoring: replace the dead_units list

eressea / server

Eressea is an automated play-by-mail strategy game server.

https://www.eressea.de/

Other

25 stars 23 forks source link

refactoring: replace the dead_units list #1066

Closed ennorehling closed 4 months ago

ennorehling commented 4 months ago

use a smaller hashmap (stb_ds) instead
fix any tests that used units after they're empty

ennorehling commented 4 months ago

Wenn ich ein eTestauswertung mit 1352 mache, stelle ich fest, das geht nicht.

Geht nicht, weil in remove_unit noch "destroy" events ausgeführt werden, und z.B. shock_handle eine Einheit in seinem Attribut hat, die bereits gelöscht wurde. Leere Einheiten sofort zu löschen ist also nicht so problemlos :-(

ennorehling commented 4 months ago

Das geht so nicht. Die Idee mit dem hashtable ist nicht verkehrt, aber die Einheiten dürfen nicht gelöscht werden (brauchen wir noch, um Messages zu generieren, usw).