[Opalvoip-svn] SF.net SVN: opalvoip:[34844] opal/trunk/src/rtp/rtp_session.cxx

[<rjo...@us...>]

Revision: 34844
          http://sourceforge.net/p/opalvoip/code/34844
Author:   rjongbloed
Date:     2016-06-13 11:08:45 +0000 (Mon, 13 Jun 2016)
Log Message:
-----------
Trace log improvement around RTP out of order resequencing.

Modified Paths:
--------------
    opal/trunk/src/rtp/rtp_session.cxx

Modified: opal/trunk/src/rtp/rtp_session.cxx
===================================================================
--- opal/trunk/src/rtp/rtp_session.cxx	2016-06-13 10:41:08 UTC (rev 34843)
+++ opal/trunk/src/rtp/rtp_session.cxx	2016-06-13 11:08:45 UTC (rev 34844)
@@ -565,7 +565,6 @@
     m_consecutiveOutOfOrderPackets = 0;
   }
   else if (sequenceDelta > SequenceReorderThreshold) {
-    PTRACE(3, &m_session, *this << "late out of order packet, got " << sequenceNumber << " expected " << expectedSequenceNumber);
     ++m_lateOutOfOrder;
     if (m_packetsLost > 0)
       --m_packetsLost; // Previously marked as lost
@@ -575,9 +574,14 @@
     if (running && ++m_lateOutOfOrderAdaptCount >= m_lateOutOfOrderAdaptMax) {
       PTimeInterval timeout = m_session.GetOutOfOrderWaitTime() + m_lateOutOfOrderAdaptBoost;
       m_session.SetOutOfOrderWaitTime(timeout);
-      PTRACE(2, &m_session, *this << " increased out of order packet timeout to " << timeout);
+      PTRACE(2, &m_session, *this << "late out of order packet:"
+                                     " got " << sequenceNumber << ", expected " << expectedSequenceNumber << ","
+                                     " increased timeout to " << setprecision(2) << timeout);
       running = false;
     }
+    else {
+      PTRACE(3, &m_session, *this << "late out of order packet: got " << sequenceNumber << ", expected " << expectedSequenceNumber);
+    }
     if (!running) {
       m_lateOutOfOrderAdaptTimer = m_lateOutOfOrderAdaptPeriod;
       m_lateOutOfOrderAdaptCount = 0;

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.