Revert 233c872.. "* Call client.Start() sunchronously. Calling thos async avoids some stuttering", pending fix for the core issue

1 files changed, 1 insertions, 10 deletions

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
index f2bcc0b..cda461c 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
             {