From: Thomas Vander Stichele <thomas@apestaart.org>
Date: Thu, 9 Feb 2006 17:04:18 +0000 (+0000)
Subject: gst-libs/gst/rtp/gstbasertppayload.c: update seqnum before setting it on the packet... 
X-Git-Tag: 1.19.3~511^2~12204
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=fdaa7a7a00e03e14919ac59c92361aef96c9ee6a;p=platform%2Fupstream%2Fgstreamer.git

gst-libs/gst/rtp/gstbasertppayload.c: update seqnum before setting it on the packet; this makes sure that the timesta...

Original commit message from CVS:

* gst-libs/gst/rtp/gstbasertppayload.c:
(gst_basertppayload_setcaps), (gst_basertppayload_push):
update seqnum before setting it on the packet; this makes sure
that the timestamp and seqnum properties match after pushing
a buffer
---

diff --git a/ChangeLog b/ChangeLog
index 7b38ba7eef..25fd512d58 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2006-02-09  Thomas Vander Stichele  <thomas at apestaart dot org>
+
+	* gst-libs/gst/rtp/gstbasertppayload.c:
+	(gst_basertppayload_setcaps), (gst_basertppayload_push):
+	  update seqnum before setting it on the packet; this makes sure
+	  that the timestamp and seqnum properties match after pushing
+	  a buffer
+
 2006-02-09  Andy Wingo  <wingo@pobox.com>
 
 	* gst-libs/gst/audio/gstringbuffer.c
diff --git a/gst-libs/gst/rtp/gstbasertppayload.c b/gst-libs/gst/rtp/gstbasertppayload.c
index f49797fa56..9b3018098b 100644
--- a/gst-libs/gst/rtp/gstbasertppayload.c
+++ b/gst-libs/gst/rtp/gstbasertppayload.c
@@ -223,6 +223,7 @@ gst_basertppayload_setcaps (GstPad * pad, GstCaps * caps)
   GstBaseRTPPayloadClass *basertppayload_class;
   gboolean ret = TRUE;
 
+  GST_DEBUG_OBJECT (pad, "setting caps %" GST_PTR_FORMAT, caps);
   basertppayload = GST_BASE_RTP_PAYLOAD (gst_pad_get_parent (pad));
   basertppayload_class = GST_BASE_RTP_PAYLOAD_GET_CLASS (basertppayload);
 
@@ -377,7 +378,10 @@ gst_basertppayload_push (GstBaseRTPPayload * payload, GstBuffer * buffer)
   gst_rtp_buffer_set_payload_type (buffer, payload->pt);
 
   /* can wrap around, which is perfectly fine */
-  gst_rtp_buffer_set_seq (buffer, payload->seqnum++);
+  /* update first, so that the property is set to the last
+   * seqnum pushed */
+  payload->seqnum++;
+  gst_rtp_buffer_set_seq (buffer, payload->seqnum);
 
   /* add our random offset to the timestamp */
   ts = payload->ts_base;