Simplified calculating the PTS offset in cPtsFixer::Fix() and fixed the overflow handling of PCR values

3 files changed, 8 insertions, 10 deletions

diff --git a/CONTRIBUTORS b/CONTRIBUTORS
index cd58e9b4..a993e76b 100644
--- a/CONTRIBUTORS
+++ b/CONTRIBUTORS
@@ -2978,6 +2978,8 @@ Sören Moch <smoch@web.de>
  packets that have a payload
  for pointing out that when adjusting the DTS values in the cutter, it hase to compensate
  for dropped B-frames
+ for simplifying calculating the PTS offset in cPtsFixer::Fix() and fixing the overflow
+ handling of PCR values
 
 Peter Münster <pmlists@free.fr>
  for fixing 'make install' to not overwrite existing configuration files
diff --git a/HISTORY b/HISTORY
index d2b82b09..68b41cf0 100644
--- a/HISTORY
+++ b/HISTORY
@@ -7356,3 +7356,5 @@ Video Disk Recorder Revision History
 - Fixed adjusting the DTS values in the cutter, to compensate for dropped B-frames
   (pointed out by Sören Moch).
 - Fixed a typo in skins.h (thanks to Lars Hanisch).
+- Simplified calculating the PTS offset in cPtsFixer::Fix() and fixed the overflow
+  handling of PCR values (thanks to Sören Moch).
diff --git a/cutter.c b/cutter.c
index 81df1869..300eef43 100644
--- a/cutter.c
+++ b/cutter.c
@@ -4,7 +4,7 @@
  * See the main source file 'vdr.c' for copyright information and
  * how to reach the author.
  *
- * $Id: cutter.c 2.20 2012/11/29 15:30:01 kls Exp $
+ * $Id: cutter.c 2.21 2012/12/02 14:30:55 kls Exp $
  */
 
 #include "cutter.h"
@@ -192,14 +192,8 @@ void cPtsFixer::Fix(uchar *Data, int Length, bool CutIn)
   // Determine the PTS offset at the beginning of each sequence (except the first one):
   if (CutIn && lastPts >= 0) {
      int64_t Pts = TsGetPts(Data, Length);
-     if (Pts >= 0) {
-        // offset is calculated so that Pts + offset results in lastPts + delta:
-        offset = Pts - PtsAdd(lastPts, delta);
-        if (offset <= 0)
-           offset = -offset;
-        else
-           offset = MAX33BIT + 1 - offset;
-        }
+     if (Pts >= 0)
+        offset = (lastPts + delta - Pts) & MAX33BIT; // offset is calculated so that Pts + offset results in lastPts + delta
      fixCounters = true;
      }
   // Keep track of the highest video PTS:
@@ -262,7 +256,7 @@ void cPtsFixer::Fix(uchar *Data, int Length, bool CutIn)
            int64_t Pcr = TsGetPcr(p);
            if (Pcr >= 0) {
               int64_t NewPcr = Pcr + offset * PCRFACTOR;
-              if (NewPcr >= MAX27MHZ)
+              if (NewPcr > MAX27MHZ)
                  NewPcr -= MAX27MHZ + 1;
               TsSetPcr(p, NewPcr);
               }