* 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

1 files changed, 8 insertions, 7 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);