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

remove-scene-viewer
Justin Clark-Casey (justincc) 2011-10-17 20:50:29 +01:00
parent 96ff2c63ed
commit 120114e96b
6 changed files with 16 additions and 28 deletions

View File

@ -1010,7 +1010,7 @@ namespace OpenSim.Framework
/// <summary>
/// Set the debug level at which packet output should be printed to console.
/// </summary>
void SetDebugPacketLevel(int newDebug);
int DebugPacketLevel { get; set; }
void InPacket(object NewPack);
void ProcessInPacket(Packet NewPack);

View File

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

View File

@ -466,7 +466,7 @@ namespace OpenSim.Region.Framework.Scenes
scenePresence.Lastname,
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
public void SetDebugPacketLevel(int newDebug)
{
}
public int DebugPacketLevel { get; set; }
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)
{

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)
{