- Blood: there's no need to store the loadsave handlers in local variables that never get used.

This commit is contained in:
Christoph Oelckers 2020-10-11 10:52:13 +02:00
parent 47e452f452
commit 3b59bf759c
16 changed files with 15 additions and 65 deletions

View file

@ -6948,11 +6948,10 @@ void ActorLoadSave::Save(void)
Write(gPost, sizeof(gPost)); Write(gPost, sizeof(gPost));
} }
static ActorLoadSave *myLoadSave;
void ActorLoadSaveConstruct(void) void ActorLoadSaveConstruct(void)
{ {
myLoadSave = new ActorLoadSave(); new ActorLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -1757,11 +1757,9 @@ void AILoadSave::Save(void)
Write(gGenDudeExtra, sizeof(gGenDudeExtra)); Write(gGenDudeExtra, sizeof(gGenDudeExtra));
} }
static AILoadSave *myLoadSave;
void AILoadSaveConstruct(void) void AILoadSaveConstruct(void)
{ {
myLoadSave = new AILoadSave(); new AILoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -606,7 +606,7 @@ void sub_76A08(spritetype *pSprite, spritetype *pSprite2, PLAYER *pPlayer) // ??
} }
} }
void DropVoodoo(int nSprite) // unused void DropVoodooCb(int nSprite) // unused
{ {
spritetype *pSprite = &sprite[nSprite]; spritetype *pSprite = &sprite[nSprite];
int nOwner = actSpriteOwnerToSpriteId(pSprite); int nOwner = actSpriteOwnerToSpriteId(pSprite);
@ -760,7 +760,7 @@ void(*gCallback[kCallbackMax])(int) =
fxPodBloodSpray, fxPodBloodSpray,
fxPodBloodSplat, fxPodBloodSplat,
LeechStateTimer, LeechStateTimer,
DropVoodoo, // unused DropVoodooCb, // unused
#ifdef NOONE_EXTENSIONS #ifdef NOONE_EXTENSIONS
callbackUniMissileBurst, // the code is in nnexts.cpp callbackUniMissileBurst, // the code is in nnexts.cpp
callbackMakeMissileBlocking, // the code is in nnexts.cpp callbackMakeMissileBlocking, // the code is in nnexts.cpp

View file

@ -289,11 +289,10 @@ void EndGameLoadSave::Save(void)
CSecretMgr gSecretMgr; CSecretMgr gSecretMgr;
CKillMgr gKillMgr; CKillMgr gKillMgr;
static EndGameLoadSave* myLoadSave;
void EndGameLoadSaveConstruct(void) void EndGameLoadSaveConstruct(void)
{ {
myLoadSave = new EndGameLoadSave(); new EndGameLoadSave();
} }

View file

@ -609,11 +609,10 @@ void EventQLoadSave::Save()
Write(bucketHead, sizeof(bucketHead)); Write(bucketHead, sizeof(bucketHead));
} }
static EventQLoadSave *myLoadSave;
void EventQLoadSaveConstruct(void) void EventQLoadSaveConstruct(void)
{ {
myLoadSave = new EventQLoadSave(); new EventQLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -257,8 +257,6 @@ class LevelsLoadSave : public LoadSave
}; };
static LevelsLoadSave *myLoadSave;
void LevelsLoadSave::Load(void) void LevelsLoadSave::Load(void)
{ {
Read(&gNextLevel, sizeof(gNextLevel)); Read(&gNextLevel, sizeof(gNextLevel));
@ -273,7 +271,7 @@ void LevelsLoadSave::Save(void)
void LevelsLoadSaveConstruct(void) void LevelsLoadSaveConstruct(void)
{ {
myLoadSave = new LevelsLoadSave(); new LevelsLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -736,9 +736,6 @@ void MyLoadSave::Save(void)
Write(skyInfo, sizeof(*skyInfo)); Write(skyInfo, sizeof(*skyInfo));
} }
static MyLoadSave *myLoadSave;
void ActorLoadSaveConstruct(void); void ActorLoadSaveConstruct(void);
void AILoadSaveConstruct(void); void AILoadSaveConstruct(void);
void EndGameLoadSaveConstruct(void); void EndGameLoadSaveConstruct(void);
@ -751,14 +748,13 @@ void SeqLoadSaveConstruct(void);
void TriggersLoadSaveConstruct(void); void TriggersLoadSaveConstruct(void);
void ViewLoadSaveConstruct(void); void ViewLoadSaveConstruct(void);
void WarpLoadSaveConstruct(void); void WarpLoadSaveConstruct(void);
void WeaponLoadSaveConstruct(void);
#ifdef NOONE_EXTENSIONS #ifdef NOONE_EXTENSIONS
void NNLoadSaveConstruct(void); void NNLoadSaveConstruct(void);
#endif #endif
void LoadSaveSetup(void) void LoadSaveSetup(void)
{ {
myLoadSave = new MyLoadSave(); new MyLoadSave();
ActorLoadSaveConstruct(); ActorLoadSaveConstruct();
AILoadSaveConstruct(); AILoadSaveConstruct();
@ -772,7 +768,6 @@ void LoadSaveSetup(void)
TriggersLoadSaveConstruct(); TriggersLoadSaveConstruct();
ViewLoadSaveConstruct(); ViewLoadSaveConstruct();
WarpLoadSaveConstruct(); WarpLoadSaveConstruct();
WeaponLoadSaveConstruct();
#ifdef NOONE_EXTENSIONS #ifdef NOONE_EXTENSIONS
NNLoadSaveConstruct(); NNLoadSaveConstruct();
#endif #endif

View file

@ -568,11 +568,9 @@ void MessagesLoadSave::Save()
Write(&bPlayerCheated, sizeof(bPlayerCheated)); Write(&bPlayerCheated, sizeof(bPlayerCheated));
} }
static MessagesLoadSave *myLoadSave;
void MessagesLoadSaveConstruct(void) void MessagesLoadSaveConstruct(void)
{ {
myLoadSave = new MessagesLoadSave(); new MessagesLoadSave();
} }
void InitCheats() void InitCheats()

View file

@ -427,8 +427,6 @@ public:
void Save(void); void Save(void);
}; };
static MirrorLoadSave *myLoadSave;
void MirrorLoadSave::Load(void) void MirrorLoadSave::Load(void)
{ {
#ifdef USE_OPENGL #ifdef USE_OPENGL
@ -472,7 +470,7 @@ void MirrorLoadSave::Save(void)
void MirrorLoadSaveConstruct(void) void MirrorLoadSaveConstruct(void)
{ {
myLoadSave = new MirrorLoadSave(); new MirrorLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -5284,11 +5284,9 @@ void NNLoadSave::Save(void)
Write(&gEventRedirectsUsed, sizeof(gEventRedirectsUsed)); Write(&gEventRedirectsUsed, sizeof(gEventRedirectsUsed));
} }
static NNLoadSave* myLoadSave;
void NNLoadSaveConstruct(void) void NNLoadSaveConstruct(void)
{ {
myLoadSave = new NNLoadSave(); new NNLoadSave();
} }
/////////////////////////////////////////////////////////////////// ///////////////////////////////////////////////////////////////////

View file

@ -2210,11 +2210,9 @@ void PlayerLoadSave::Save(void)
#endif #endif
} }
static PlayerLoadSave *myLoadSave;
void PlayerLoadSaveConstruct(void) void PlayerLoadSaveConstruct(void)
{ {
myLoadSave = new PlayerLoadSave(); new PlayerLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -33,8 +33,6 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
BEGIN_BLD_NS BEGIN_BLD_NS
#define gotpic blafasl
int nPrecacheCount; int nPrecacheCount;
void fxPrecache(HitList &hits); void fxPrecache(HitList &hits);

View file

@ -2309,11 +2309,9 @@ void TriggersLoadSave::Save()
Write(basePath, sizeof(basePath)); Write(basePath, sizeof(basePath));
} }
static TriggersLoadSave *myLoadSave;
void TriggersLoadSaveConstruct(void) void TriggersLoadSaveConstruct(void)
{ {
myLoadSave = new TriggersLoadSave(); new TriggersLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -1064,8 +1064,6 @@ public:
void Save(void); void Save(void);
}; };
static ViewLoadSave *myLoadSave;
static int messageTime; static int messageTime;
static char message[256]; static char message[256];
@ -1095,7 +1093,7 @@ void ViewLoadSave::Save(void)
void ViewLoadSaveConstruct(void) void ViewLoadSaveConstruct(void)
{ {
myLoadSave = new ViewLoadSave(); new ViewLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -323,11 +323,9 @@ void WarpLoadSave::Save()
Write(gLowerLink, sizeof(gLowerLink)); Write(gLowerLink, sizeof(gLowerLink));
} }
static WarpLoadSave *myLoadSave;
void WarpLoadSaveConstruct(void) void WarpLoadSaveConstruct(void)
{ {
myLoadSave = new WarpLoadSave(); new WarpLoadSave();
} }
END_BLD_NS END_BLD_NS

View file

@ -2545,26 +2545,4 @@ void sub_51340(spritetype *pMissile, int a2)
} }
} }
class WeaponLoadSave : public LoadSave
{
public:
virtual void Load();
virtual void Save();
};
void WeaponLoadSave::Load()
{
}
void WeaponLoadSave::Save()
{
}
static WeaponLoadSave *myLoadSave;
void WeaponLoadSaveConstruct(void)
{
myLoadSave = new WeaponLoadSave();
}
END_BLD_NS END_BLD_NS