refactor: Make IClientAPI.DebugPacketFormat a property rather than a setter without a getter

0.7.2-post-fixes
Justin Clark-Casey (justincc) 2011-10-17 20:50:29 +01:00
parent 2a7a5c4c90
commit 31ac7571f4
6 changed files with 16 additions and 28 deletions

View File

@ -1007,11 +1007,11 @@ namespace OpenSim.Framework
event MuteListEntryRemove OnRemoveMuteListEntry; event MuteListEntryRemove OnRemoveMuteListEntry;
event GodlikeMessage onGodlikeMessage; event GodlikeMessage onGodlikeMessage;
event GodUpdateRegionInfoUpdate OnGodUpdateRegionInfoUpdate; event GodUpdateRegionInfoUpdate OnGodUpdateRegionInfoUpdate;
/// <summary> /// <summary>
/// Set the debug level at which packet output should be printed to console. /// Set the debug level at which packet output should be printed to console.
/// </summary> /// </summary>
void SetDebugPacketLevel(int newDebug); int DebugPacketLevel { get; set; }
void InPacket(object NewPack); void InPacket(object NewPack);
void ProcessInPacket(Packet NewPack); void ProcessInPacket(Packet NewPack);

View File

@ -64,7 +64,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
/// <value> /// <value>
/// Debug packet level. See OpenSim.RegisterConsoleCommands() for more details. /// Debug packet level. See OpenSim.RegisterConsoleCommands() for more details.
/// </value> /// </value>
protected int m_debugPacketLevel = 0; public int DebugPacketLevel { get; set; }
#region Events #region Events
@ -477,11 +477,6 @@ namespace OpenSim.Region.ClientStack.LindenUDP
RegisterLocalPacketHandlers(); RegisterLocalPacketHandlers();
} }
public void SetDebugPacketLevel(int newDebug)
{
m_debugPacketLevel = newDebug;
}
#region Client Methods #region Client Methods
/// <summary> /// <summary>
@ -11594,25 +11589,25 @@ namespace OpenSim.Region.ClientStack.LindenUDP
/// provide your own method.</param> /// provide your own method.</param>
protected void OutPacket(Packet packet, ThrottleOutPacketType throttlePacketType, bool doAutomaticSplitting, UnackedPacketMethod method) protected void OutPacket(Packet packet, ThrottleOutPacketType throttlePacketType, bool doAutomaticSplitting, UnackedPacketMethod method)
{ {
if (m_debugPacketLevel > 0) if (DebugPacketLevel > 0)
{ {
bool logPacket = true; bool logPacket = true;
if (m_debugPacketLevel <= 255 if (DebugPacketLevel <= 255
&& (packet.Type == PacketType.SimStats || packet.Type == PacketType.SimulatorViewerTimeMessage)) && (packet.Type == PacketType.SimStats || packet.Type == PacketType.SimulatorViewerTimeMessage))
logPacket = false; logPacket = false;
if (m_debugPacketLevel <= 200 if (DebugPacketLevel <= 200
&& (packet.Type == PacketType.ImagePacket && (packet.Type == PacketType.ImagePacket
|| packet.Type == PacketType.ImageData || packet.Type == PacketType.ImageData
|| packet.Type == PacketType.LayerData || packet.Type == PacketType.LayerData
|| packet.Type == PacketType.CoarseLocationUpdate)) || packet.Type == PacketType.CoarseLocationUpdate))
logPacket = false; logPacket = false;
if (m_debugPacketLevel <= 100 && (packet.Type == PacketType.AvatarAnimation || packet.Type == PacketType.ViewerEffect)) if (DebugPacketLevel <= 100 && (packet.Type == PacketType.AvatarAnimation || packet.Type == PacketType.ViewerEffect))
logPacket = false; logPacket = false;
if (m_debugPacketLevel <= 50 && packet.Type == PacketType.ImprovedTerseObjectUpdate) if (DebugPacketLevel <= 50 && packet.Type == PacketType.ImprovedTerseObjectUpdate)
logPacket = false; logPacket = false;
if (logPacket) if (logPacket)
@ -11657,17 +11652,17 @@ namespace OpenSim.Region.ClientStack.LindenUDP
/// <param name="Pack">OpenMetaverse.packet</param> /// <param name="Pack">OpenMetaverse.packet</param>
public void ProcessInPacket(Packet packet) public void ProcessInPacket(Packet packet)
{ {
if (m_debugPacketLevel > 0) if (DebugPacketLevel > 0)
{ {
bool outputPacket = true; bool outputPacket = true;
if (m_debugPacketLevel <= 255 && packet.Type == PacketType.AgentUpdate) if (DebugPacketLevel <= 255 && packet.Type == PacketType.AgentUpdate)
outputPacket = false; outputPacket = false;
if (m_debugPacketLevel <= 200 && packet.Type == PacketType.RequestImage) if (DebugPacketLevel <= 200 && packet.Type == PacketType.RequestImage)
outputPacket = false; outputPacket = false;
if (m_debugPacketLevel <= 100 && (packet.Type == PacketType.ViewerEffect || packet.Type == PacketType.AgentAnimation)) if (DebugPacketLevel <= 100 && (packet.Type == PacketType.ViewerEffect || packet.Type == PacketType.AgentAnimation))
outputPacket = false; outputPacket = false;
if (outputPacket) if (outputPacket)

View File

@ -466,7 +466,7 @@ namespace OpenSim.Region.Framework.Scenes
scenePresence.Lastname, scenePresence.Lastname,
newDebug); newDebug);
scenePresence.ControllingClient.SetDebugPacketLevel(newDebug); scenePresence.ControllingClient.DebugPacketLevel = newDebug;
} }
}); });
} }

View File

@ -866,10 +866,7 @@ namespace OpenSim.Region.OptionalModules.Agent.InternetRelayClientView.Server
#pragma warning restore 67 #pragma warning restore 67
public void SetDebugPacketLevel(int newDebug) public int DebugPacketLevel { get; set; }
{
}
public void InPacket(object NewPack) public void InPacket(object NewPack)
{ {

View File

@ -823,9 +823,7 @@ namespace OpenSim.Region.OptionalModules.World.NPC
{ {
} }
public void SetDebugPacketLevel(int newDebug) public int DebugPacketLevel { get; set; }
{
}
public void InPacket(object NewPack) public void InPacket(object NewPack)
{ {

View File

@ -873,9 +873,7 @@ namespace OpenSim.Tests.Common.Mock
{ {
} }
public void SetDebugPacketLevel(int newDebug) public int DebugPacketLevel { get; set; }
{
}
public void InPacket(object NewPack) public void InPacket(object NewPack)
{ {