* minor: Change around more debugging messages

4 files changed, 31 insertions, 13 deletions

diff --git a/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs b/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs
index bb7406f..6bcc77e 100644
--- a/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs
+++ b/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs
@@ -338,7 +338,9 @@ namespace OpenSim.Region.Environment.Scenes.Hypergrid
                         if (avatar.Scene.NeedSceneCacheClear(avatar.UUID) || isHyperLink)
                         {
                             m_commsProvider.UserProfileCacheService.RemoveUser(avatar.UUID);
-                            m_log.InfoFormat("[HGSceneCommService]: User {0} is going to another region, profile cache removed", avatar.UUID);
+                            m_log.DebugFormat(
+                                "[HGSceneCommService]: User {0} is going to another region, profile cache removed",
+                                avatar.UUID);                            
                         }
                     }
                     else
diff --git a/OpenSim/Region/Environment/Scenes/Scene.cs b/OpenSim/Region/Environment/Scenes/Scene.cs
index 11655e9..4ff4c0e 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.cs
@@ -2759,15 +2759,15 @@ namespace OpenSim.Region.Environment.Scenes
         /// <param name="agent"></param>
         public void NewUserConnection(AgentCircuitData agent)
         {
-            // Don't disable this log messages - it's too useful
+            // Don't disable this log message - it's too helpful
             m_log.DebugFormat(
-                "[CONNECTION SETUP]: Scene {0} told of incoming client {1} {2} {3} (circuit code {4})", 
+                "[CONNECTION BEGIN]: Scene {0} told of incoming client {1} {2} {3} (circuit code {4})", 
                 RegionInfo.RegionName, agent.firstname, agent.lastname, agent.AgentID, agent.circuitcode);
             
             if (m_regInfo.EstateSettings.IsBanned(agent.AgentID))
             {
                 m_log.WarnFormat(
-               "[CONNECTION SETUP]: Denied access to: {0} at {1} because the user is on the region banlist",
+               "[CONNECTION BEGIN]: Denied access to: {0} at {1} because the user is on the region banlist",
                agent.AgentID, RegionInfo.RegionName);
             }
 
@@ -2778,11 +2778,22 @@ namespace OpenSim.Region.Environment.Scenes
             ScenePresence sp = m_sceneGraph.GetScenePresence(agent.AgentID);
             if (sp != null)
             {
-                m_log.DebugFormat(
-                    "[CONNECTION SETUP]: Updated existing agent {0} in {1}", agent.AgentID, RegionInfo.RegionName);
+                if (sp.IsChildAgent)
+                    m_log.DebugFormat(
+                        "[CONNECTION BEGIN]: Incoming client for {0} is existing child agent {1}", 
+                        RegionInfo.RegionName, agent.AgentID);
+                else
+                    m_log.DebugFormat(
+                        "[CONNECTION BEGIN]: Incoming client for {0} is existing root agent {1}", 
+                        RegionInfo.RegionName, agent.AgentID);
+                    
+//                m_log.DebugFormat(
+//                    "[CONNECTION BEGIN]: Updated existing agent {0} in {1}", agent.AgentID, RegionInfo.RegionName);
+                
                 sp.AdjustKnownSeeds();
                 sp.AbsolutePosition = agent.startpos;
                 m_authenticateHandler.AddNewCircuit(agent.circuitcode, agent);
+                
                 return;
             }
 
@@ -2813,7 +2824,7 @@ namespace OpenSim.Region.Environment.Scenes
             else
             {
                 m_log.WarnFormat(
-                    "[CONNECTION SETUP]: We couldn't find a User Info record for {0}.  This is usually an indication that the UUID we're looking up is invalid", agent.AgentID);
+                    "[CONNECTION BEGIN]: We couldn't find a User Info record for {0}.  This is usually an indication that the UUID we're looking up is invalid", agent.AgentID);
             }
         }
 
diff --git a/OpenSim/Region/Environment/Scenes/SceneCommunicationService.cs b/OpenSim/Region/Environment/Scenes/SceneCommunicationService.cs
index 7997fba..ab4e7d8 100644
--- a/OpenSim/Region/Environment/Scenes/SceneCommunicationService.cs
+++ b/OpenSim/Region/Environment/Scenes/SceneCommunicationService.cs
@@ -916,10 +916,10 @@ namespace OpenSim.Region.Environment.Scenes
                         if (avatar.Scene.NeedSceneCacheClear(avatar.UUID))
                         {
                             m_commsProvider.UserProfileCacheService.RemoveUser(avatar.UUID);
-                            m_log.InfoFormat("User {0} is going to another region, profile cache removed", avatar.UUID);
+                            m_log.DebugFormat(
+                                "[SCENE COMMUNICATION SERVICE]: User {0} is going to another region, profile cache removed",
+                                avatar.UUID);
                         }
-
-
                     }
                     else
                     {
diff --git a/OpenSim/Region/Environment/Scenes/ScenePresence.cs b/OpenSim/Region/Environment/Scenes/ScenePresence.cs
index 1a3971d..00e5548 100644
--- a/OpenSim/Region/Environment/Scenes/ScenePresence.cs
+++ b/OpenSim/Region/Environment/Scenes/ScenePresence.cs
@@ -2483,7 +2483,8 @@ namespace OpenSim.Region.Environment.Scenes
                     if (m_scene.NeedSceneCacheClear(UUID))
                     {
                         m_scene.CommsManager.UserProfileCacheService.RemoveUser(UUID);
-                        m_log.InfoFormat("[AVATAR]: User {0} is going to another region, profile cache removed", UUID);
+                        m_log.DebugFormat(
+                            "[SCENE PRESENCE]: User {0} is going to another region, profile cache removed", UUID);
                     }
                 }
                 else
@@ -2510,7 +2511,9 @@ namespace OpenSim.Region.Environment.Scenes
         public void CloseChildAgents(uint newRegionX, uint newRegionY)
         {
             List<ulong> byebyeRegions = new List<ulong>();
-            m_log.DebugFormat("[AVATAR]: Closing child agents. Checking {0} regions in {1}", m_knownChildRegions.Keys.Count, Scene.RegionInfo.RegionName);
+            m_log.DebugFormat(
+                "[SCENE PRESENCE]: Closing child agents. Checking {0} regions in {1}", 
+                m_knownChildRegions.Keys.Count, Scene.RegionInfo.RegionName);
             //DumpKnownRegions();
 
             lock (m_knownChildRegions)
@@ -2534,11 +2537,13 @@ namespace OpenSim.Region.Environment.Scenes
                     }
                 }
             }
+            
             if (byebyeRegions.Count > 0)
             {
-                m_log.Info("[AVATAR]: Closing " + byebyeRegions.Count + " child agents");
+                m_log.Debug("[SCENE PRESENCE]: Closing " + byebyeRegions.Count + " child agents");
                 m_scene.SceneGridService.SendCloseChildAgentConnections(m_controllingClient.AgentId, byebyeRegions);
             }
+            
             foreach (ulong handle in byebyeRegions)
             {
                 RemoveNeighbourRegion(handle);