[Bug 732265] vaapidecode: h264: add support for decoding MVC base views only

[GStreamer (GNOME Bugzilla)]

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

--- Comment #25 from Orestis Floros <orestisf1993 at gmail.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #24)
> (In reply to Orestis Floros from comment #22)
> > > @@ +1222,3 @@
> > > +  }
> > > +
> > > +  if (base_only && !have_mvc && have_high) {
> > >
> > > If va backend doesn't profile mvc profiles but has high profile, I would set
> > > base_only as TRUE mandatory.
> > 
> > Ok. This means that gstvaapidecode doesn't need to get the value of
> > base_only since it won't use it anywhere. It need to set base_only
> > accordingly though.
> 
> I would use base_only property if the use doesn't want MVC/SVC support on
> purpose.

Yes. I am saying that what I am doing in attachment 356380 won't be needed at
all since vaapidecode won't need to read the property, only vaapih264dec.

> (In reply to Orestis Floros from comment #23)
> > 
> > It would also mean that h264parse won't be able to use the changes in bug
> > 732267 because vaapih264dec would fake MVC/SVC support.
> 
> h264parse change is useful for avdec_h264 users or any other decoder that
> might not handle MVC/SVC streams.

Ok. The problem here might be that we will always lie about SVC support without
the user asking for it and with no way to disable it. Could this be problematic
behavior for some cases?

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