[Bug 723352] codecparsers_h264: Framerate estimation enhancement

[GStreamer (GNOME Bugzilla)]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #328484|none                        |committed
             status|                            |

--- Comment #15 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 328484
  --> https://bugzilla.gnome.org/attachment.cgi?id=328484
smoothstreaming: update fps calculation for h264 codec parser API changes

Pushed this, don't see how it can possibly make anything worse.

Only found one working sample stream with H264 but that one yields 0/1.

commit 7d46d67c59eab6cfda24d43220dfba2b0f9131d3
Author: Tim-Philipp Müller <tim at centricular.com>
Date:   Wed May 25 10:59:00 2016 +0100

    smoothstreaming: update fps calculation for h264 codec parser API changes

    Use new gst_h264_video_calculate_framerate() API instead of fps_n/fps_d
    fields in SPS struct which are to be removed.

    Apparently H264 content in MSS is always non-interlaced/progressive,
    so we can just pass 0 for field_pic_flag and don't need to parse any
    slice headers first if there's no external signalling. But even if
    that's not the case the new code is not worse than the existing code.

    https://msdn.microsoft.com/en-us/library/cc189080%28VS.95%29.aspx

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

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