From 17412389f3cbc4a98c3fb11963ebec821fcd5e11 Mon Sep 17 00:00:00 2001
From: Tom Grimshaw <tom@meta7.com>
Date: Sat, 17 Jul 2010 14:34:35 -0700
Subject: [PATCH] Revert 233c872.. "* Call client.Start() sunchronously.
 Calling thos async avoids some stuttering", pending fix for the core issue

---
 OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
index f2bcc0b48f..cda461c4b8 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
@@ -900,16 +900,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
 
                 // Start the IClientAPI
                 // Spin it off so that it doesn't clog up the LLUDPServer
-                //Util.FireAndForget(delegate(object o) { client.Start(); });
-
-                // NOTE: DO NOT CALL THIS ASYNCHRONOUSLY!!!!!
-                // This method will ultimately cause the modules to hook
-                // client events in OnNewClient. If they can't do this
-                // before further packets are processed, packets WILL BE LOST.
-                // This includes the all-important EconomyDataRequest!
-                // So using FireAndForget here WILL screw up money. Badly.
-                // You have been warned!
-                client.Start();
+                Util.FireAndForget(delegate(object o) { client.Start(); });
             }
             else
             {