After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 747845 - rtpsource: Try to keep clock-rate and payload type updated for sender RTCP too
rtpsource: Try to keep clock-rate and payload type updated for sender RTCP too
Status: RESOLVED DUPLICATE of bug 747394
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
unspecified
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-04-14 11:39 UTC by Sebastian Dröge (slomo)
Modified: 2015-04-20 07:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
rtpsource: Try to keep clock-rate and payload type updated for sender RTCP too (1.35 KB, patch)
2015-04-14 11:39 UTC, Sebastian Dröge (slomo)
none Details | Review
rtpsource: Try to keep clock-rate and payload type updated for sender RTCP too (2.38 KB, patch)
2015-04-14 11:57 UTC, Sebastian Dröge (slomo)
none Details | Review
rtprtxsend: Copy over timestamps from the orignal buffers to the RTX buffers (904 bytes, patch)
2015-04-14 11:57 UTC, Sebastian Dröge (slomo)
none Details | Review

Description Sebastian Dröge (slomo) 2015-04-14 11:39:08 UTC
See commit message. The only downside is now that we always call that for RTX
streams, and before this we never emitted request-pt-map for senders. Only for
receivers. So if someone implements RTX now and does not handle this, the
signal will be emitted for every single packet that is sent.

The alternative would be to always change the caps in rtxsend whenever it
sends a RTX packet, and back to the other caps when it sends a normal packet.


This fixes the "no clock-rate, cannot interpolate rtp time" warning in
rtp_source_get_new_sr() for the RTX streams.
Comment 1 Sebastian Dröge (slomo) 2015-04-14 11:39:12 UTC
Created attachment 301528 [details] [review]
rtpsource: Try to keep clock-rate and payload type updated for sender RTCP too

For RTX we will get packets with different payload types, and only for the
main stream we have the clock-rate and everything inside the caps. For the RTX
stream we will have to ask. The clock-rate is later needed for properly
creating sender reports.
Comment 2 Sebastian Dröge (slomo) 2015-04-14 11:57:20 UTC
Created attachment 301531 [details] [review]
rtpsource: Try to keep clock-rate and payload type updated for sender RTCP too

For RTX we will get packets with different payload types, and only for the
main stream we have the clock-rate and everything inside the caps. For the RTX
stream we will have to ask. The clock-rate is later needed for properly
creating sender reports.
Comment 3 Sebastian Dröge (slomo) 2015-04-14 11:57:26 UTC
Created attachment 301532 [details] [review]
rtprtxsend: Copy over timestamps from the orignal buffers to the RTX buffers
Comment 4 Sebastian Dröge (slomo) 2015-04-14 12:10:39 UTC

*** This bug has been marked as a duplicate of bug 747394 ***