From 1d8821ccb2783463720380d7d61adda427074dad Mon Sep 17 00:00:00 2001 From: diva Date: Sun, 18 Jan 2009 01:45:22 +0000 Subject: Getting rid of the CheckRegion call during TPs. This seems to be not just useless, but sometimes problematic (mantis #2999). Initial tests indicate that this call is not necessary. Let's see if this stands in the wild. --- .../Scenes/Hypergrid/HGSceneCommunicationService.cs | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) (limited to 'OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs') diff --git a/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs b/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs index 6bcc77e..a652609 100644 --- a/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs +++ b/OpenSim/Region/Environment/Scenes/Hypergrid/HGSceneCommunicationService.cs @@ -70,7 +70,7 @@ namespace OpenSim.Region.Environment.Scenes.Hypergrid if (!avatar.Scene.Permissions.CanTeleport(avatar.UUID)) return; - bool destRegionUp = false; + bool destRegionUp = true; IEventQueue eq = avatar.Scene.RequestModuleInterface(); @@ -137,17 +137,6 @@ namespace OpenSim.Region.Environment.Scenes.Hypergrid avatar.ControllingClient.SendTeleportLocationStart(); - if (reg.RemotingAddress != "" && reg.RemotingPort != 0) - { - // region is remote. see if it is up - destRegionUp = m_commsProvider.InterRegion.CheckRegion(reg.RemotingAddress, reg.RemotingPort); - } - else - { - // assume local regions are always up - destRegionUp = true; - } - // Let's do DNS resolution only once in this process, please! // This may be a costly operation. The reg.ExternalEndPoint field is not a passive field, // it's actually doing a lot of work. -- cgit v1.1