From f97aeab9163fb24bee16a0a85328a8d177b046fb Mon Sep 17 00:00:00 2001
From: Teravus Ovares <teravus@gmail.com>
Date: Thu, 22 Nov 2007 21:40:53 +0000
Subject: [PATCH] * Quelled a Log format exception in the logging routines. *
 Tweaked some esoteric throttle settings * Removed AgentThrottle from the
 unsupported packet list.

---
 OpenSim/Framework/Console/LogBase.cs            | 17 +++++++++++++++--
 .../ClientStack/ClientView.ProcessPackets.cs    | 11 ++++++++---
 OpenSim/Region/ClientStack/ClientView.cs        |  4 ++--
 3 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/OpenSim/Framework/Console/LogBase.cs b/OpenSim/Framework/Console/LogBase.cs
index 6651e5d061..ec3c5bc720 100644
--- a/OpenSim/Framework/Console/LogBase.cs
+++ b/OpenSim/Framework/Console/LogBase.cs
@@ -238,9 +238,16 @@ namespace OpenSim.Framework.Console
             {
                 string now = DateTime.Now.ToString("[MM-dd hh:mm:ss] ");
                 Log.Write(now);
-                Log.WriteLine(format, args);
-                Log.Flush();
+                try
+                {
+                    Log.WriteLine(format, args);
+                    Log.Flush();
+                }
 
+                catch (FormatException FE)
+                {
+                    System.Console.WriteLine(args);
+                }
                 System.Console.Write(now);
                 try
                 {
@@ -255,6 +262,12 @@ namespace OpenSim.Framework.Console
                     // Some older systems dont support coloured text.
                     System.Console.WriteLine(format, args);
                 }
+                 catch (System.FormatException)
+                {
+                    // Some older systems dont support coloured text.
+                    System.Console.WriteLine(args);
+                }
+                
                 return;
             }
         }
diff --git a/OpenSim/Region/ClientStack/ClientView.ProcessPackets.cs b/OpenSim/Region/ClientStack/ClientView.ProcessPackets.cs
index 93c12a680c..8c4dba65ad 100644
--- a/OpenSim/Region/ClientStack/ClientView.ProcessPackets.cs
+++ b/OpenSim/Region/ClientStack/ClientView.ProcessPackets.cs
@@ -783,7 +783,7 @@ namespace OpenSim.Region.ClientStack
 
                     case PacketType.AgentThrottle:
 
-                        OpenSim.Framework.Console.MainLog.Instance.Verbose("CLIENT", "unhandled packet " + Pack.ToString());
+                        //OpenSim.Framework.Console.MainLog.Instance.Verbose("CLIENT", "unhandled packet " + Pack.ToString());
 
                         AgentThrottlePacket atpack = (AgentThrottlePacket)Pack;
 
@@ -825,6 +825,7 @@ namespace OpenSim.Region.ClientStack
                         tAsset = (int)BitConverter.ToSingle(throttle, j);
 
                         tall = tResend + tLand + tWind + tCloud + tTask + tTexture + tAsset;
+                        /*
                         OpenSim.Framework.Console.MainLog.Instance.Verbose("CLIENT", "Client AgentThrottle - Got throttle:resendbytes=" + tResend +
                             " landbytes=" + tLand +
                             " windbytes=" + tWind +
@@ -833,6 +834,8 @@ namespace OpenSim.Region.ClientStack
                             " texturebytes=" + tTexture +
                             " Assetbytes=" + tAsset +
                             " Allbytes=" + tall);
+                         */
+
                         // Total Sanity
                         // Make sure that the client sent sane total values.
 
@@ -896,7 +899,7 @@ namespace OpenSim.Region.ClientStack
                             if (tAsset < AssetthrottleMin)
                                 AssetthrottleOutbound = AssetthrottleMin;
 
-                            OpenSim.Framework.Console.MainLog.Instance.Verbose("THROTTLE", "Using:resendbytes=" + ResendthrottleOutbound +
+                          /*  OpenSim.Framework.Console.MainLog.Instance.Verbose("THROTTLE", "Using:resendbytes=" + ResendthrottleOutbound +
                             " landbytes=" + LandthrottleOutbound +
                             " windbytes=" + WindthrottleOutbound +
                             " cloudbytes=" + CloudthrottleOutbound +
@@ -904,6 +907,7 @@ namespace OpenSim.Region.ClientStack
                             " texturebytes=" + TexturethrottleOutbound +
                             " Assetbytes=" + AssetthrottleOutbound +
                             " Allbytes=" + tall);
+                           */
                         }
                         else
                         {
@@ -1006,7 +1010,7 @@ namespace OpenSim.Region.ClientStack
                                 if (tAssetResult < AssetthrottleMin)
                                     AssetthrottleOutbound = AssetthrottleMin;
 
-                                OpenSim.Framework.Console.MainLog.Instance.Verbose("THROTTLE", "Using:resendbytes=" + ResendthrottleOutbound +
+                                /* OpenSim.Framework.Console.MainLog.Instance.Verbose("THROTTLE", "Using:resendbytes=" + ResendthrottleOutbound +
                                     " landbytes=" + LandthrottleOutbound +
                                     " windbytes=" + WindthrottleOutbound +
                                     " cloudbytes=" + CloudthrottleOutbound +
@@ -1014,6 +1018,7 @@ namespace OpenSim.Region.ClientStack
                                     " texturebytes=" + TexturethrottleOutbound +
                                     " Assetbytes=" + AssetthrottleOutbound +
                                     " Allbytes=" + tall);
+                                  */
 
                             }
                             else
diff --git a/OpenSim/Region/ClientStack/ClientView.cs b/OpenSim/Region/ClientStack/ClientView.cs
index 411883b2b6..b4441ec8f0 100644
--- a/OpenSim/Region/ClientStack/ClientView.cs
+++ b/OpenSim/Region/ClientStack/ClientView.cs
@@ -96,7 +96,7 @@ namespace OpenSim.Region.ClientStack
         // TODO: Make this variable. Lower throttle on un-ack. Raise over time
 
         // All throttle times and number of bytes are calculated by dividing by this value
-        private int throttleTimeDivisor = 5;
+        private int throttleTimeDivisor = 7;
 
         private int throttletimems = 1000;
 
@@ -208,7 +208,7 @@ namespace OpenSim.Region.ClientStack
             // The General overhead of the UDP protocol gets sent to the queue un-throttled by this
             // so This'll pick up about around the right time.
 
-            int MaxThrottleLoops = 5550; // 50*7 packets can be dequeued at once.
+            int MaxThrottleLoops = 4550; // 50*7 packets can be dequeued at once.
             int throttleLoops = 0;
 
             // We're going to dequeue all of the saved up packets until