[Bug 752807] h265parse: Fix sticky event mishandling when stream does not have VPS

[GStreamer (GNOME Bugzilla)]

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

Vineeth <vineeth.tm at samsung.com> changed:

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

--- Comment #3 from Vineeth <vineeth.tm at samsung.com> ---
Created attachment 308421
  --> https://bugzilla.gnome.org/attachment.cgi?id=308421&action=edit
avoid checking VPS NAL which is optional

changed the commit message.

The reason i feel VPS is optional is
In the code it is commented stating
    case GST_H265_NAL_VPS:
      /* It is not mandatory to have VPS in the stream. But it might
       * be needed for other extensions like svc */


and there is one other check in code, where they check for presence of only SPS
and PPS
    if (nalu.type == GST_H265_NAL_VPS ||
        nalu.type == GST_H265_NAL_SPS ||
        nalu.type == GST_H265_NAL_PPS ||
        (h265parse->have_sps && h265parse->have_pps))



And i did some googling and found few places where they were discussing about
it being optional.
http://forum.doom9.org/archive/index.php/t-167081.html 
"1) VPS is optional - agreed"

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