OpenMeridian / Meridian59

The MMORPG Meridian 59 - Server 103
openmeridian.org
Other
143 stars 75 forks source link

Destroy Before Free #1395

Open ememisya opened 3 years ago

ememisya commented 3 years ago

Might I suggest something like

   if (IsInterpreting())
      SendBlakodMessage(object_id,DELETE_MSG,0,NULL);
   else
      SendTopLevelBlakodMessage(object_id,DELETE_MSG,0,NULL);

   /* now remove object */

Prepended to

https://github.com/OpenMeridian/Meridian59/blob/35db18ae00c7713da4d6bb4564454eab93c68bce/blakserv/object.c#L330

This would also better address https://github.com/OpenMeridian/Meridian59/issues/1391 , thoughts?