Added some error handling (and console output) to BaseHttpServer.

a few other bits of refactoring.

3 files changed, 11 insertions, 15 deletions

diff --git a/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs b/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs
index b6c373c..c33fdcd 100644
--- a/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs
+++ b/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs
@@ -12,7 +12,7 @@ namespace OpenSim.Region.Environment.Modules
     public class AvatarFactoryModule : IAvatarFactory
     {
         private Scene m_scene = null;
-        private Dictionary<LLUUID, AvatarWearing> m_avatarsClothes = new Dictionary<LLUUID, AvatarWearing>();
+        private Dictionary<LLUUID, AvatarAppearance> m_avatarsClothes = new Dictionary<LLUUID, AvatarAppearance>();
 
         public bool TryGetIntialAvatarAppearance(LLUUID avatarId, out AvatarWearable[] wearables,
                                                  out byte[] visualParams)
@@ -20,13 +20,13 @@ namespace OpenSim.Region.Environment.Modules
             if (!m_avatarsClothes.ContainsKey(avatarId))
             {
                 GetDefaultAvatarAppearance(out wearables, out visualParams);
-                AvatarWearing wearing = new AvatarWearing(wearables);
+                AvatarAppearance wearing = new AvatarAppearance(wearables);
                 m_avatarsClothes[avatarId] = wearing;
                 return true;
             }
             else
             {
-                visualParams = SetDefaultVisualParams();
+                visualParams = GetDefaultVisualParams();
                 wearables = m_avatarsClothes[avatarId].IsWearing;
                 return true;
             }
@@ -91,7 +91,7 @@ namespace OpenSim.Region.Environment.Modules
                                 //Tempoaray dictionary storage. This is be storing to a database
                                 if (m_avatarsClothes.ContainsKey(clientView.AgentId))
                                 {
-                                    AvatarWearing avWearing = m_avatarsClothes[clientView.AgentId];
+                                    AvatarAppearance avWearing = m_avatarsClothes[clientView.AgentId];
                                     avWearing.IsWearing[wear.Type].AssetID = assetId;
                                     avWearing.IsWearing[wear.Type].ItemID = wear.ItemID;
                                 }
@@ -103,12 +103,11 @@ namespace OpenSim.Region.Environment.Modules
 
         public static void GetDefaultAvatarAppearance(out AvatarWearable[] wearables, out byte[] visualParams)
         {
-            visualParams = SetDefaultVisualParams();
-
+            visualParams = GetDefaultVisualParams();
             wearables = AvatarWearable.DefaultWearables;
         }
 
-        private static byte[] SetDefaultVisualParams()
+        private static byte[] GetDefaultVisualParams()
         {
             byte[] visualParams;
             visualParams = new byte[218];
@@ -119,11 +118,12 @@ namespace OpenSim.Region.Environment.Modules
             return visualParams;
         }
 
-        public class AvatarWearing
+        public class AvatarAppearance
         {
             public AvatarWearable[] IsWearing;
+            public byte[] VisualParams;
 
-            public AvatarWearing()
+            public AvatarAppearance()
             {
                 IsWearing = new AvatarWearable[13];
                 for (int i = 0; i < 13; i++)
@@ -132,7 +132,7 @@ namespace OpenSim.Region.Environment.Modules
                 }
             }
 
-            public AvatarWearing(AvatarWearable[] wearing)
+            public AvatarAppearance(AvatarWearable[] wearing)
             {
                 if (wearing.Length == 13)
                 {
diff --git a/OpenSim/Region/Environment/Scenes/Scene.PacketHandlers.cs b/OpenSim/Region/Environment/Scenes/Scene.PacketHandlers.cs
index 435022b..745345d 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.PacketHandlers.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.PacketHandlers.cs
@@ -190,9 +190,6 @@ namespace OpenSim.Region.Environment.Scenes
             }
             replyPacket.Data = searchData;
             client.SendAvatarPickerReply(replyPacket);
-            
-            
         }
-        
     }
 }
\ No newline at end of file
diff --git a/OpenSim/Region/Environment/Scenes/ScenePresence.cs b/OpenSim/Region/Environment/Scenes/ScenePresence.cs
index 46079bb..2ef27fd 100644
--- a/OpenSim/Region/Environment/Scenes/ScenePresence.cs
+++ b/OpenSim/Region/Environment/Scenes/ScenePresence.cs
@@ -511,8 +511,7 @@ namespace OpenSim.Region.Environment.Scenes
             for (int i = 0; i < visualParam.Length; i++)
             {
                 m_visualParams[i] = visualParam[i].ParamValue;
-                //OpenSim.Framework.Console.MainLog.Instance.Verbose("CLIENT", "VisualData[" + i.ToString() + "]: " + visualParam[i].ParamValue.ToString() + "m");
-                
+                //OpenSim.Framework.Console.MainLog.Instance.Verbose("CLIENT", "VisualData[" + i.ToString() + "]: " + visualParam[i].ParamValue.ToString() + "m");  
             }
             
             // Teravus : Nifty AV Height Getting Maaaaagical formula.  Oh how we love turning 0-255 into meters.