[Bug 762509] H.264 decoder stores too many pictures in the DPB before output

[GStreamer (GNOME Bugzilla)]

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

sreerenj <bsreerenj at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #330336|none                        |rejected
             status|                            |

--- Comment #10 from sreerenj <bsreerenj at gmail.com> ---
Review of attachment 330336:
 --> (https://bugzilla.gnome.org/review?bug=762509&attachment=330336)

I am rejecting this patch because of 2 reasons:

1: It is breaking the MVC playback.
You can try with reference streams MVCDS-1.264 or any of the MVCDS samples.

2: As per the spec (E.2.1)  The value of max_dec_frame_buffering shall be
*greater than or equal* to max_num_ref_frames (== sps->num_ref_frames) . And if
the syntax element
max_dec_frame_buffering is not present(== in theory we consider this as no VUI
header, but still approximating the value of max_dec_frame_buffering for
finding the dpb size),
the value should be inferred to be equal to MaxDpbFrames (which is equal to 9
in the mentioned stream).

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