[Bug 700749] gstrtph264pay/depay: Add framerate and optional framesize SDP attribute to payloaded caps

[GStreamer (bugzilla.gnome.org)]

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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk

--- Comment #21 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-05-22 22:23:03 UTC ---
I would just like to remark that there are two possible locale-related problems
with the float parsing/generating: (a) on the sender side and (b) on the
receiver side. We may only control one part of the equation. My patch tried to
also accommodate the case where someone created the SDP or whatever using
printf using their own locale (which happened to use a comma as decimal
separator). This is wrong of course, but then if we always errored out whenever
something doesn't adhere 100% to the spec we could just shut down everything ;)
True story btw IIRC, not just some "just in case" thing.

When *creating* the string from the float, you should always use a decimal
point irrespective of locale, so g_ascii_formatd() or g_ascii_dtostr() should
be sufficient.

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