diff options
author | Justin Clarke Casey | 2008-08-12 19:00:13 +0000 |
---|---|---|
committer | Justin Clarke Casey | 2008-08-12 19:00:13 +0000 |
commit | 4e555b87f31c0e82b6120564cd6d8e2b41a3b8bf (patch) | |
tree | 6aab22c68ef6fbc8262567cb0cdaa511ca965b54 /OpenSim/Region/Environment/Scenes/InnerScene.cs | |
parent | Thanks, lulurun, for a patch that addresses inventory problems that occur (diff) | |
download | opensim-SC-4e555b87f31c0e82b6120564cd6d8e2b41a3b8bf.zip opensim-SC-4e555b87f31c0e82b6120564cd6d8e2b41a3b8bf.tar.gz opensim-SC-4e555b87f31c0e82b6120564cd6d8e2b41a3b8bf.tar.bz2 opensim-SC-4e555b87f31c0e82b6120564cd6d8e2b41a3b8bf.tar.xz |
* Stop warnings about non existent scene presences/entities being removed on client log off
* This is being done by preventing close from being called twice on child agent closure (nres which would have been thrown are being swallowed).
* However, it should be possible to do much better cleanup on this code in the future
Diffstat (limited to '')
-rw-r--r-- | OpenSim/Region/Environment/Scenes/InnerScene.cs | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/OpenSim/Region/Environment/Scenes/InnerScene.cs b/OpenSim/Region/Environment/Scenes/InnerScene.cs index c4e4919..cdc05ee 100644 --- a/OpenSim/Region/Environment/Scenes/InnerScene.cs +++ b/OpenSim/Region/Environment/Scenes/InnerScene.cs | |||
@@ -584,26 +584,26 @@ namespace OpenSim.Region.Environment.Scenes | |||
584 | { | 584 | { |
585 | lock (Entities) | 585 | lock (Entities) |
586 | { | 586 | { |
587 | if (Entities.Remove(agentID)) | 587 | if (!Entities.Remove(agentID)) |
588 | { | ||
589 | //m_log.InfoFormat("[SCENE] Removed scene presence {0} from entities list", agentID); | ||
590 | } | ||
591 | else | ||
592 | { | 588 | { |
593 | m_log.WarnFormat("[SCENE] Tried to remove non-existent scene presence with agent ID {0} from scene Entities list", agentID); | 589 | m_log.WarnFormat("[SCENE] Tried to remove non-existent scene presence with agent ID {0} from scene Entities list", agentID); |
594 | } | 590 | } |
591 | // else | ||
592 | // { | ||
593 | // m_log.InfoFormat("[SCENE] Removed scene presence {0} from entities list", agentID); | ||
594 | // } | ||
595 | } | 595 | } |
596 | 596 | ||
597 | lock (ScenePresences) | 597 | lock (ScenePresences) |
598 | { | 598 | { |
599 | if (ScenePresences.Remove(agentID)) | 599 | if (!ScenePresences.Remove(agentID)) |
600 | { | ||
601 | //m_log.InfoFormat("[SCENE] Removed scene presence {0}", agentID); | ||
602 | } | ||
603 | else | ||
604 | { | 600 | { |
605 | m_log.WarnFormat("[SCENE] Tried to remove non-existent scene presence with agent ID {0} from scene ScenePresences list", agentID); | 601 | m_log.WarnFormat("[SCENE] Tried to remove non-existent scene presence with agent ID {0} from scene ScenePresences list", agentID); |
606 | } | 602 | } |
603 | // else | ||
604 | // { | ||
605 | // m_log.InfoFormat("[SCENE] Removed scene presence {0} from scene presences list", agentID); | ||
606 | // } | ||
607 | } | 607 | } |
608 | } | 608 | } |
609 | 609 | ||