[gstreamer-bugs] [Bug 639292] New: remove pt value from selected rtp pay/depayloaders

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=639292
  GStreamer | gst-plugins-good | 0.10.26

           Summary: remove pt value from selected rtp pay/depayloaders
    Classification: Desktop
           Product: GStreamer
           Version: 0.10.26
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: marc.leeman at gmail.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


Created an attachment (id=178115)
 View: https://bugzilla.gnome.org/attachment.cgi?id=178115
 Review: https://bugzilla.gnome.org/review?bug=639292&attachment=178115

put payload in sink caps in comment

This patch removes the payload requirement in the caps of

- rtpmp4vpay
- rtpmp4vdepay
- rtph264pay
- rtph264depay
- rtph263pay
- rtph263depay
- rtph263ppay
- rtph263pdepay

Rationale:

encoding name is required in the caps and is a better fit for autoplugging than
the pt value. Hardware manufacturers have a bad habit of skimming through RFCs
and in this case; use unassigned numbers for encoders instead of dynamic
numbers.

However, I could not really find any reference in the RFC that I had a look at
that explicitly forbids this (though it is poor practice since the number are
unassigned).

In essence, this patch will add support for a lot of Bosch hardware encoders
without breaking autoplugging (we are running this fix a couple of months)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.