cWorld: Fixed scheduler.

Fixes #1534.
Added a test case into the Debuggers plugin.

MCServer/Plugins/Debuggers/Debuggers.lua

@@ -10,9 +10,6 @@ g_ShowFoodStats = false;  -- When true, each player's food stats are sent to the
 
 
 function Initialize(Plugin)
-	Plugin:SetName("Debuggers")
-	Plugin:SetVersion(1)
-	
 	--[[
 	-- Test multiple hook handlers:
 	cPluginManager.AddHook(cPluginManager.HOOK_TICK,                         OnTick1);
@@ -69,6 +66,8 @@ function Initialize(Plugin)
 	PM:BindCommand("/pickups", "debuggers", HandlePickups,         "- Spawns random pickups around you");
 	PM:BindCommand("/poof",    "debuggers", HandlePoof,            "- Nudges pickups close to you away from you");
 	
+	PM:BindConsoleCommand("sched", HandleConsoleSchedule, "Tests the world scheduling");
+
 	Plugin:AddWebTab("Debuggers",  HandleRequest_Debuggers)
 	Plugin:AddWebTab("StressTest", HandleRequest_StressTest)
 
@@ -1630,3 +1629,17 @@ end
 
 
 
+
+function HandleConsoleSchedule(a_Split)
+	LOG("Scheduling a task for 2 seconds in the future")
+	cRoot:Get():GetDefaultWorld():ScheduleTask(40,
+		function ()
+			LOG("Scheduled function is called.")
+		end
+	)
+	return true, "Task scheduled"
+end
+
+
+
+

src/World.cpp

@@ -1068,15 +1068,24 @@ void cWorld::TickScheduledTasks(void)
 	{
 		cCSLock Lock(m_CSScheduledTasks);
 		auto WorldAge = m_WorldAge;
-		std::move(
-			m_ScheduledTasks.begin(),
-			std::find_if(
-				m_ScheduledTasks.begin(),
-				m_ScheduledTasks.end(),
-				[WorldAge] (cScheduledTaskPtr & Task) { return (Task->m_TargetTick < WorldAge);}
-			),
-			std::back_inserter(Tasks)
-		);
+
+		// Move all the due tasks from m_ScheduledTasks into Tasks:
+		for (auto itr = m_ScheduledTasks.begin(); itr != m_ScheduledTasks.end();)  // Cannot use range-basd for, we're modifying the container
+		{
+			if ((*itr)->m_TargetTick < WorldAge)
+			{
+				auto next = itr;
+				++next;
+				Tasks.push_back(std::move(*itr));
+				m_ScheduledTasks.erase(itr);
+				itr = next;
+			}
+			else
+			{
+				// All the eligible tasks have been moved, bail out now
+				break;
+			}
+		}
 	}
 
 	// Execute and delete each task: