From 391f2822c8f6da7d6ffa8114817a8baf2aa265d5 Mon Sep 17 00:00:00 2001
From: Adam <Adam@anope.org>
Date: Thu, 14 Feb 2013 20:57:40 -0500
Subject: This Serialize::Destroy method isn't actually needed anymore. Fixes
 weirdness from a few Serializable items we had on the stack. Added a comment
 about why operator< in Reference fails.

---
 src/serialize.cpp | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

(limited to 'src/serialize.cpp')

diff --git a/src/serialize.cpp b/src/serialize.cpp
index 4f0068a48..8828965b5 100644
--- a/src/serialize.cpp
+++ b/src/serialize.cpp
@@ -67,6 +67,8 @@ Serializable::Serializable(const Serializable &other) : last_commit(NULL), last_
 
 Serializable::~Serializable()
 {
+	FOREACH_MOD(I_OnSerializableDestruct, OnSerializableDestruct(this));
+
 	SerializableItems->erase(this->s_iter);
 	delete last_commit;
 }
@@ -76,16 +78,6 @@ Serializable &Serializable::operator=(const Serializable &)
 	return *this;
 }
 
-void Serializable::Destroy()
-{
-	if (!this)
-		return;
-
-	FOREACH_MOD(I_OnSerializableDestruct, OnSerializableDestruct(this));
-
-	delete this;
-}
-
 void Serializable::QueueUpdate()
 {
 	/* Schedule updater */
-- 
cgit