Comment out SCRAMBLE_REMOVED

- Scramble can help point out memory leaks from mobjs
  never being freed
- However it can also cause false positive crashes, since
  there is sometimes a 1 or 2 tic window between when
  P_RemoveMobj is called and all pointers to that mobj are
  set to NULL
- Presumably we have code that NULL checks but does not
  check P_MobjWasRemoved and such instances may be hard to
  find so it's just easier to not scramble
This commit is contained in:
James R 2024-03-03 00:32:38 -08:00
parent 6dbc093516
commit db7892d15a

View file

@ -11290,7 +11290,7 @@ tic_t itemrespawntime[ITEMQUESIZE];
size_t iquehead, iquetail;
#ifdef PARANOIA
#define SCRAMBLE_REMOVED // Force debug build to crash when Removed mobj is accessed
//#define SCRAMBLE_REMOVED // Force debug build to crash when Removed mobj is accessed
#endif
void P_RemoveMobj(mobj_t *mobj)
{