[Bug 758515] dashdemux : suggestion for setting the framerate info at dashdemux.

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=758515

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk
 Attachment #316072|none                        |needs-work
             status|                            |

--- Comment #2 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 316072
  --> https://bugzilla.gnome.org/attachment.cgi?id=316072
This patch suggested for the setting framerate at dashdemux.

Instead of this GstFrameRate helper structure (which is being leaked in
multiple cases here), just set gint fps_den = 0, fps_num = 0; and do:

  static gboolean
  gst_mpd_client_get_video_stream_framerate (GstActiveStream * stream,
      gint * fps_den, gint *fps_num)

Also, the framerate should probably only be set on the caps if it could be
retrieved.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.