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

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #21 from Sebastian Rasmussen <sebras at hotmail.com> 2013-05-23 19:02:52 UTC ---
(In reply to comment #20)
> Vs comment #19, I think all that does is reject "height is there but is 0"..

yes, exactly.

> That said, framerate 0/N is valid and should be ignored, not rejected.

but my framerate checks are on the form (num < 0 || denom <= 0) which means
that if num == 0 and denom == 1 the expression evaluates to FALSE which means
that the framerate will not be considered invalid (the jump is not taken).

so any negative numerator or denominator would be invalid, as would a
denominator which is equal to zero. I believe this to be correct.

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