[Bug 700747] rtsp-sdp: Parse framerate and optional framesize attribute from payloaded caps and put in SDP

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=700747
  GStreamer | gst-rtsp-server | git

--- Comment #6 from Sebastian Rasmussen <sebras at hotmail.com> 2013-05-21 07:58:52 UTC ---
> I'm a bit concerned about adding the framerate to the caps as a string because
> that makes it non-negotiable. It probably makes sense to make it a GstFraction
> type, since it doesn't go into rtpmap, then it can be made negotiable.

How about the x-dimensions already existing in rtpjpegpay/depay? I was trying
to adhere to that way of doing it. Maybe this is viewed as non-negotiable which
means that... 

> Probably the same can be said of framesize (maybe put it as width and height of
> type int).. Although this is a bit less problematic as it is defined as purely
> declarative in the RFC.

...one can follow this line of reasoning. I agree though that both attributes
probably should be treated the same. Should I additionally do something about
x-dimensions..? If I supply width/height in the caps then there is little
reason to supply x-dimensions as a string..?

PS. Oh, and using strings doesn't _strictly_ mean making something
non-negotiable, but it is certainly more difficult so I do get your point. ;)

-- 
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.