- restored calling OnRegister for event handlers after loading a savegame

It should be said in no uncertain terms that OnRegister operates on an uninitialized level so it should only be used for setting up the registering process of the event handler itself and nothing else - not even the event handler's data!!!

src/events.cpp

@@ -46,6 +46,13 @@
 EventManager staticEventManager;
 EventManager eventManager;
 
+void EventManager::CallOnRegister()
+{
+	for (DStaticEventHandler* handler = FirstEventHandler; handler; handler = handler->next)
+	{
+		handler->OnRegister();
+	}
+}
 
 bool EventManager::RegisterHandler(DStaticEventHandler* handler)
 {

src/events.h

@@ -240,6 +240,8 @@ struct EventManager
 	~EventManager() { Shutdown(); }
 	bool ShouldCallStatic(bool forplay);
 
+	// for use after loading a savegame. The old handler explicitly reinstalled all handlers instead of doing a list deserialization which resulted in OnRegister being called even when a save was loaded.
+	void CallOnRegister();
 	// register
 	bool RegisterHandler(DStaticEventHandler* handler);
 	// unregister

src/p_saveg.cpp

@@ -994,6 +994,7 @@ void FLevelLocals::Serialize(FSerializer &arc, bool hubload)
 	// [ZZ] serialize events
 	arc("firstevent", localEventManager->FirstEventHandler)
 		("lastevent", localEventManager->LastEventHandler);
+	localEventManager->CallOnRegister();
 	Thinkers.SerializeThinkers(arc, hubload);
 	arc("polyobjs", Polyobjects);
 	SerializeSubsectors(arc, "subsectors");