Changeset 887 for pjproject

Timestamp:

Jan 20, 2007 5:14:24 AM (18 years ago)

Author:

bennylp

Message:

Fixed ticket #70: Frame timestamp not propagated correctly in PJMEDIA

Location:

pjproject/trunk

Files:

• pjlib/include/pj/os.h (modified) (3 diffs)
• pjmedia/src/pjmedia/conference.c (modified) (6 diffs)

pjproject/trunk/pjlib/include/pj/os.h

@@ -922 +922 @@
 
 /**
+ * Set timestamp from 32bit values.
+ * @param t         The timestamp to be set.
+ * @param hi        The high 32bit part.
+ * @param lo        The low 32bit part.
+ */
+PJ_INLINE(void) pj_set_timestamp32(pj_timestamp *t, pj_uint32_t hi,
+                                   pj_uint32_t lo)
+{
+    t->u32.hi = hi;
+    t->u32.lo = lo;
+}
+
+/**
  * Add timestamp t2 to t1.
  * @param t1        t1.
@@ -934 +947 @@
     t1->u32.hi += t2->u32.hi;
     t1->u32.lo += t2->u32.lo;
+    if (t1->u32.lo < old)
+        ++t1->u32.hi;
+#endif
+}
+
+/**
+ * Add timestamp t2 to t1.
+ * @param t1        t1.
+ * @param t2        t2.
+ */
+PJ_INLINE(void) pj_add_timestamp32(pj_timestamp *t1, pj_uint32_t t2)
+{
+#if PJ_HAS_INT64
+    t1->u64 += t2;
+#else
+    pj_uint32_t old = t1->u32.lo;
+    t1->u32.lo += t2;
     if (t1->u32.lo < old)
         ++t1->u32.hi;
@@ -960 +990 @@
 
 /**
+ * Substract timestamp t2 from t1.
+ * @param t1        t1.
+ * @param t2        t2.
+ */
+PJ_INLINE(void) pj_sub_timestamp32(pj_timestamp *t1, pj_uint32_t t2)
+{
+#if PJ_HAS_INT64
+    t1->u64 -= t2;
+#else
+    if (t1->u32.lo >= t2)
+        t1->u32.lo -= t2;
+    else {
+        t1->u32.lo -= t2;
+        --t1->u32.hi;
+    }
+#endif
+}
+
+/**
+ * Get the timestamp difference between t2 and t1 (that is t2 minus t1),
+ * and return a 32bit signed integer difference.
+ */
+PJ_INLINE(pj_int32_t) pj_timestamp_diff32(const pj_timestamp *t1,
+                                          const pj_timestamp *t2)
+{
+    /* Be careful with the signess (I think!) */
+#if PJ_HAS_INT64
+    pj_int64_t diff = t2->u64 - t1->u64;
+    return (pj_int32_t) diff;
+#else
+    pj_int32 diff = t2->u32.lo - t1->u32.lo;
+    return diff;
+#endif
+}
+
+
+/**
  * Calculate the elapsed time, and store it in pj_time_val.
  * This function calculates the elapsed time using highest precision

pjproject/trunk/pjmedia/src/pjmedia/conference.c

@@ -1312 +1312 @@
  */
 static pj_status_t write_port(pjmedia_conf *conf, struct conf_port *cport,
-                              pj_uint32_t timestamp, 
+                              const pj_timestamp *timestamp, 
                               pjmedia_frame_type *frm_type)
 {
     pj_int16_t *buf;
-    unsigned j;
+    unsigned j, ts;
     pj_status_t status;
 
@@ -1332 +1332 @@
         pjmedia_frame frame;
 
+        /* Adjust the timestamp */
+        frame.timestamp.u64 = timestamp->u64 * cport->clock_rate /
+                                conf->clock_rate;
         frame.type = PJMEDIA_FRAME_TYPE_NONE;
         frame.buf = NULL;
@@ -1458 +1461 @@
             frame.buf = (pj_int16_t*)cport->mix_buf;
             frame.size = conf->samples_per_frame * BYTES_PER_SAMPLE;
-            frame.timestamp.u64 = timestamp;
+            /* No need to adjust timestamp, port has the same
+             * clock rate as conference bridge 
+             */
+            frame.timestamp = *timestamp;
 
             TRACE_((THIS_FILE, "put_frame %.*s, count=%d", 
@@ -1492 +1498 @@
     /* Transmit while we have enough frame in the tx_buf. */
     status = PJ_SUCCESS;
+    ts = 0;
     while (cport->tx_buf_count >= cport->samples_per_frame &&
            status == PJ_SUCCESS) 
@@ -1506 +1513 @@
             frame.buf = cport->tx_buf;
             frame.size = cport->samples_per_frame * BYTES_PER_SAMPLE;
-            frame.timestamp.u64 = timestamp;
+            /* Adjust timestamp as port may have different clock rate
+             * than the bridge.
+             */
+            frame.timestamp.u64 = timestamp->u64 * cport->clock_rate /
+                                  conf->clock_rate;
+
+            /* Add timestamp for individual frame */
+            frame.timestamp.u64 += ts;
+            ts += cport->samples_per_frame;
 
             TRACE_((THIS_FILE, "put_frame %.*s, count=%d", 
@@ -1747 +1762 @@
         ++ci;
 
-        status = write_port( conf, conf_port, frame->timestamp.u32.lo,
+        status = write_port( conf, conf_port, &frame->timestamp,
                              &frm_type);
         if (status != PJ_SUCCESS) {