Don't dynamically allocate commands in modules anymore, instead made them members of modules. This means the commands are automatically destructed when the module is unloaded. Cleans up some old ugly code.

1 files changed, 5 insertions, 3 deletions

diff --git a/modules/core/ns_identify.cpp b/modules/core/ns_identify.cpp
index bdf6beffa..db9a70a55 100644
--- a/modules/core/ns_identify.cpp
+++ b/modules/core/ns_identify.cpp
@@ -113,14 +113,16 @@ class CommandNSIdentify : public Command
 
 class NSIdentify : public Module
 {
+	CommandNSIdentify commandnsidentify, commandnsid;
+
  public:
-	NSIdentify(const Anope::string &modname, const Anope::string &creator) : Module(modname, creator)
+	NSIdentify(const Anope::string &modname, const Anope::string &creator) : Module(modname, creator), commandnsidentify("IDENTIFY"), commandnsid("ID")
 	{
 		this->SetAuthor("Anope");
 		this->SetType(CORE);
 
-		this->AddCommand(NickServ, new CommandNSIdentify("IDENTIFY"));
-		this->AddCommand(NickServ, new CommandNSIdentify("ID"));
+		this->AddCommand(NickServ, &commandnsidentify);
+		this->AddCommand(NickServ, &commandnsid);
 	}
 };