[Bug 695728] x264enc plugin has a divide by zero error when framerate is "0/1"

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=695728
  GStreamer | gst-plugins-ugly | 1.0.1

tcdgreenwood changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #238739|0                           |1
        is obsolete|                            |

--- Comment #3 from tcdgreenwood at hotmail.com 2013-03-14 13:46:45 UTC ---
Created an attachment (id=238873)
 View: https://bugzilla.gnome.org/attachment.cgi?id=238873
 Review: https://bugzilla.gnome.org/review?bug=695728&attachment=238873

Patch for issue reworked to remove C++ style comments

I've reworked the comments in the patch and checked two pipelines:
gst-launch-1.0 videotestsrc ! 'video/x-raw,height=480,width=640,framerate=15/1'
! x264enc ! avdec_h264 ! videoconvert ! osxvideosink
and:
gst-launch-1.0 videotestsrc ! 'video/x-raw,height=640,width=480,framerate=15/1'
! vp8enc ! rtpvp8pay ! rtpvp8depay ! vp8dec ! queue ! x264enc ! avdec_h264 !
videoconvert ! osxvideosink
It seems to be working fine.  Please let me know if there is anything else I
can do to help or if the patch needs any more work.  Thank you for helping me.

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