[Bug 732266] vaapidecode: h264: add support for decoding SVC base layers only

[GStreamer (GNOME Bugzilla)]

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

--- Comment #39 from sreerenj <bsreerenj at gmail.com> ---
(In reply to Orestis Floros from comment #37)
> (In reply to sreerenj from comment #36)
> > Here the spec clearly saying that "SPS should be replaced by another SPS
> > with the same id" and "SUBSET_SPS should be replaced by another SUBSET_SPS
> > with the same id".
> >
> > Am I misinterpreting something here? :)
> 
> I think we don't disagree here since you say:
> 
> > Hm, I am afraid we need some serious changes in
> > codecparser/videoparser/decoders since the GstH264NalParser is not keeping
> > separate array for sps and subset sps.
> 
> Yes, my solution is a bit hacky.
> 
> > Is it the only sample which exhibiting this behavior?
> 
> With some quick scripting I found that 137 out of 258 SVC *.264 conformance
> bitstreams exhibit this behavior but 0 out of 34 of the MVC ones.

Anyway changing nal_parser with a separate array for subset_sps will shake the
implementation here and there.
For the base-only scenario, I am fine with your patch which is avoiding the
override.

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