ogb912/opensim
1
0
Fork 0
mirror of https://github.com/opensim/opensim.git synced 2026-08-05 17:05:55 +08:00
Code Issues Packages Projects Releases Wiki Activity

Set IClientAPI.IsActive = false early on client removal due to ack timeout rather than using IsLoggingOut flag.

Browse Source 
IsActive is more appropriate since unack timeout is not due to voluntary logout.
This is in line with operations such as manual kick that do not set the IsLoggingOut flag.
It's also slightly better race-wise since it reduces the chance of this operation clashing with another reason for client deactivation (e.g. manual kick).
This commit is contained in:
Justin Clark-Casey (justincc)
2012-06-12 02:16:36 +01:00
parent c89db34fc4
commit b099f26376
2 changed files with 37 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
OpenSim/Framework/IClientAPI.cs
View File

@@ -741,22 +741,19 @@ namespace OpenSim.Framework
string Name { get; }
/// <summary>
/// True if the client is active (sending and receiving new UDP messages). False if the client is closing.
/// True if the client is active (sending and receiving new UDP messages). False if the client is being closed.
/// </summary>
bool IsActive { get; set; }
/// <summary>
/// Set if the client is closing due to a logout request or because of too much time since last ack.
/// Set if the client is closing due to a logout request
/// </summary>
/// <remarks>
/// Do not use this flag if you want to know if the client is closing, since it will not be set in other
/// circumstances (e.g. if a child agent is closed or the agent is kicked off the simulator). Use IsActive
/// instead.
/// instead with a IClientAPI.SceneAgent.IsChildAgent check if necessary.
///
/// Only set for root agents.
///
/// TODO: Too much time since last ack should probably be a separate property, or possibly part of a state
/// machine.
/// </remarks>
bool IsLoggingOut { get; set; }