* refactor: remove unnecessary packet null check in LLClientView

* if a packet was really null it would be caught by the general exception handler at the top of the client thread, which would also provide more information and attempt a clean 
shutdown

2 files changed, 8 insertions, 8 deletions

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
index a55ce1c..bd67f1e 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
@@ -645,23 +645,24 @@ namespace OpenSim.Region.ClientStack.LindenUDP
             }
         }
 
+        /// <summary>
+        /// Main packet processing loop for the UDP component of the client session.  Both incoming and outgoing
+        /// packets are processed here. 
+        /// </summary>
         protected virtual void ClientLoop()
         {
-            m_log.Info("[CLIENT]: Entered main packet processing loop");
+            m_log.DebugFormat(
+                "[CLIENT]: Entered main packet processing loop for {0} {1}", FirstName, LastName);
 
             while (true)
             {
                 LLQueItem nextPacket = m_PacketHandler.PacketQueue.Dequeue();
-                if (nextPacket == null)
-                {
-                    m_log.Error("Got a NULL packet in Client Loop, bailing out of our client loop");
-                    break;
-                }
+                
                 if (nextPacket.Incoming)
                 {
                     DebugPacket("IN", nextPacket.Packet);
                     m_PacketHandler.ProcessInPacket(nextPacket);
-                }
+                }                
                 else
                 {
                     DebugPacket("OUT", nextPacket.Packet);
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs b/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs
index 1e369ec..32b4bc1 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLPacketQueue.cs
@@ -135,7 +135,6 @@ namespace OpenSim.Region.ClientStack.LindenUDP
 
         /* STANDARD QUEUE MANIPULATION INTERFACES */
 
-
         public void Enqueue(LLQueItem item)
         {
             if (!m_enabled)