m_check, m_namesx, m_timedbans Minor cleanup

3 files changed, 2 insertions, 9 deletions

diff --git a/src/modules/m_check.cpp b/src/modules/m_check.cpp
index 994745422..07276445f 100644
--- a/src/modules/m_check.cpp
+++ b/src/modules/m_check.cpp
@@ -42,7 +42,7 @@ class CommandCheck : public Command
 		return std::string(timebuf);
 	}
 
-	void dumpExt(User* user, std::string checkstr, Extensible* ext)
+	void dumpExt(User* user, const std::string& checkstr, Extensible* ext)
 	{
 		std::stringstream dumpkeys;
 		for(Extensible::ExtensibleStore::const_iterator i = ext->GetExtList().begin(); i != ext->GetExtList().end(); i++)
@@ -199,7 +199,6 @@ class CommandCheck : public Command
 			}
 
 			irc::modestacker modestack(true);
-			std::string letter_b("b");
 			for(BanList::iterator b = targchan->bans.begin(); b != targchan->bans.end(); ++b)
 			{
 				modestack.Push('b', b->data);
diff --git a/src/modules/m_namesx.cpp b/src/modules/m_namesx.cpp
index 535f6ec10..e2b3efce3 100644
--- a/src/modules/m_namesx.cpp
+++ b/src/modules/m_namesx.cpp
@@ -105,7 +105,7 @@ class ModuleNamesX : public Module
 
 		// Try to find this channel
 		std::string channame = line.substr(a, b-a);
-		Channel* chan = ServerInstance->FindChan(channame.c_str());
+		Channel* chan = ServerInstance->FindChan(channame);
 		if (!chan)
 			return;
 
diff --git a/src/modules/m_timedbans.cpp b/src/modules/m_timedbans.cpp
index 472e79ca4..c639d2039 100644
--- a/src/modules/m_timedbans.cpp
+++ b/src/modules/m_timedbans.cpp
@@ -113,16 +113,10 @@ class ModuleTimedBans : public Module
 		: cmd(this)
 	{
 		ServerInstance->AddCommand(&cmd);
-		TimedBanList.clear();
 		Implementation eventlist[] = { I_OnDelBan, I_OnBackgroundTimer };
 		ServerInstance->Modules->Attach(eventlist, this, 2);
 	}
 
-	virtual ~ModuleTimedBans()
-	{
-		TimedBanList.clear();
-	}
-
 	virtual ModResult OnDelBan(User* source, Channel* chan, const std::string &banmask)
 	{
 		irc::string listitem = banmask.c_str();