Remove the time_t parameter from Tick and TickTimers.

Most uses of Tick do not use the parameter and even when it is used
it is always the same as CurTime.

include/modules/dns.h

@@ -173,7 +173,7 @@ namespace DNS
 		/** Used to time out the query, xalls OnError and lets the TimerManager
 		 * delete this request.
 		 */
-		void Tick(time_t) override
+		void Tick() override
 		{
 			Log(LOG_DEBUG_2) << "Resolver: timeout for query " << this->name;
 			Query rr(*this);

include/timers.h

@@ -83,7 +83,7 @@ public:
 	/** Called when the timer ticks
 	 * This should be overridden with something useful
 	 */
-	virtual void Tick(time_t ctime) = 0;
+	virtual void Tick() = 0;
 };
 
 /** This class manages sets of Timers, and triggers them at their defined times.
@@ -109,7 +109,7 @@ public:
 	/** Tick all pending timers
 	 * @param ctime The current time
 	 */
-	static void TickTimers(time_t ctime = Anope::CurTime);
+	static void TickTimers();
 
 	/** Deletes all timers owned by the given module
 	 */

modules/botserv/bs_kick.cpp

@@ -1010,7 +1010,7 @@ public:
 	{
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		Log(LOG_DEBUG) << "bs_main: Running bandata purger";
 

modules/botserv/bs_set.cpp

@@ -83,7 +83,7 @@ public:
 		{
 		}
 
-		void Tick(time_t) override
+		void Tick() override
 		{
 			Channel *c = Channel::Find(chname);
 			if (c)

modules/chanserv/chanserv.cpp

@@ -75,7 +75,7 @@ public:
 			/** Called when the delay is up
 			 * @param The current time
 			 */
-			void Tick(time_t) override
+			void Tick() override
 			{
 				if (!c)
 					return;

modules/chanserv/cs_ban.cpp

@@ -30,7 +30,7 @@ public:
 	{
 	}
 
-	void Tick(time_t ctime) override
+	void Tick() override
 	{
 		Channel *c = Channel::Find(this->channel);
 		if (c)

modules/dns.cpp

@@ -493,7 +493,9 @@ public:
 		}
 
 		/* Times out after a few seconds */
-		void Tick(time_t) override { }
+		void Tick() override
+		{
+		}
 
 		void Reply(Packet *p) override
 		{
@@ -968,7 +970,7 @@ public:
 		return serial;
 	}
 
-	void Tick(time_t now) override
+	void Tick() override
 	{
 		Log(LOG_DEBUG_2) << "Resolver: Purging DNS cache";
 
@@ -979,7 +981,7 @@ public:
 			it_next = it;
 			++it_next;
 
-			if (req.created + static_cast<time_t>(req.ttl) < now)
+			if (req.created + static_cast<time_t>(req.ttl) < Anope::CurTime)
 				this->cache.erase(it);
 		}
 	}

modules/httpd.cpp

@@ -300,7 +300,7 @@ public:
 	{
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		while (!this->clients.empty())
 		{

modules/nickserv/nickserv.cpp

@@ -50,7 +50,7 @@ public:
 		return na;
 	}
 
-	void Tick(time_t t) override
+	void Tick() override
 	{
 		if (!u || !na)
 			return;
@@ -78,7 +78,7 @@ public:
 		n->Extend<bool>("HELD");
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		if (na)
 			na->Shrink<bool>("HELD");
@@ -121,7 +121,9 @@ public:
 		NickServReleases.erase(this->nick);
 	}
 
-	void Tick(time_t t) override { }
+	void Tick() override
+	{
+	}
 };
 
 class NickServCore final

modules/operserv/os_defcon.cpp

@@ -123,7 +123,7 @@ public:
 		timeout = NULL;
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		if (DConfig.defaultlevel != level)
 		{

modules/proxyscan.cpp

@@ -216,7 +216,7 @@ class ModuleProxyScan final
 		{
 		}
 
-		void Tick(time_t) override
+		void Tick() override
 		{
 			for (std::set<ProxyConnect *>::iterator it = ProxyConnect::proxies.begin(), it_end = ProxyConnect::proxies.end(); it != it_end;)
 			{

modules/sasl.cpp

@@ -340,7 +340,7 @@ public:
 		this->SendMessage(session, "M", buf.empty() ? "" : buf.substr(1));
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		for (std::map<Anope::string, Session *>::iterator it = sessions.begin(); it != sessions.end();)
 		{

src/main.cpp

@@ -50,7 +50,7 @@ public:
 	{
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		Anope::SaveDatabases();
 	}
@@ -65,7 +65,7 @@ public:
 	{
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		FOREACH_MOD(OnExpireTick, ());
 	}
@@ -176,7 +176,7 @@ int main(int ac, char **av, char **envp)
 		/* Process timers */
 		if (Anope::CurTime - last_check >= Config->TimeoutCheck)
 		{
-			TimerManager::TickTimers(Anope::CurTime);
+			TimerManager::TickTimers();
 			last_check = Anope::CurTime;
 		}
 

src/timers.cpp

@@ -89,20 +89,20 @@ void TimerManager::DelTimer(Timer *t)
 	}
 }
 
-void TimerManager::TickTimers(time_t ctime)
+void TimerManager::TickTimers()
 {
 	while (!Timers.empty())
 	{
 		std::multimap<time_t, Timer *>::iterator it = Timers.begin();
 		Timer *t = it->second;
 
-		if (t->GetTimer() > ctime)
+		if (t->GetTimer() > Anope::CurTime)
 			break;
 
-		t->Tick(ctime);
+		t->Tick();
 
 		if (t->GetRepeat())
-			t->SetTimer(ctime + t->GetSecs());
+			t->SetTimer(Anope::CurTime + t->GetSecs());
 		else
 			delete t;
 	}

src/uplink.cpp

@@ -26,7 +26,7 @@ public:
 	{
 	}
 
-	void Tick(time_t) override
+	void Tick() override
 	{
 		try
 		{