* This reverts a change where m_regions[RegionHandle] was replaced instead of removed in LocalBackEndServices It used to be like the change above a long long time ago but we found that it caused intermittent neighbor connectivity issues. I added comments above it so that it's looked at more closely next time.
parent
225067d8b6
commit
b4355e4564
|
@ -102,8 +102,13 @@ namespace OpenSim.Region.Communications.Local
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
// Already in our list, so the region went dead and restarted.
|
// Already in our list, so the region went dead and restarted.
|
||||||
// replace the old regioninfo
|
// don't replace the old regioninfo.. this might be a locking issue.. however we need to
|
||||||
m_regions[regionInfo.RegionHandle] = regionInfo;
|
// remove it and let it add normally below or we get extremely strange and intermittant
|
||||||
|
// connectivity errors.
|
||||||
|
// Don't change this line below to 'm_regions[regionInfo.RegionHandle] = regionInfo' unless you
|
||||||
|
// *REALLY* know what you are doing here.
|
||||||
|
m_regions.Remove(regionInfo.RegionHandle);
|
||||||
|
|
||||||
m_log.Warn("[INTERREGION STANDALONE]: Region registered twice. Region went down and came back up.");
|
m_log.Warn("[INTERREGION STANDALONE]: Region registered twice. Region went down and came back up.");
|
||||||
|
|
||||||
RegionCommsListener regionHost = new RegionCommsListener();
|
RegionCommsListener regionHost = new RegionCommsListener();
|
||||||
|
|
Loading…
Reference in New Issue