[Bug 796984] vaapih265enc: Support hevc encode low latency B frame.

[GStreamer (GNOME Bugzilla)]

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

--- Comment #11 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
(In reply to Fei from comment #10)
> (In reply to Víctor Manuel Jáquez Leal from comment #9)
> > Review of attachment 373414 [details] [review] [review]:
> > 
> > lgtm
> > 
> > though we would need to reach a consensus on the property prefix and if the
> > property is required at all
> 
> Hi Victor,
> Just as Haihao's comment, low delay B frame is not intel specific, so I
> think it's better to remove prefix Intel. Otherwise, it will confused. How
> do you think of this ?

I agree it is not Intel exclusive, but still it is very driver backend
specific. What would happen if users of other backends enables it breaking
their uses cases?

We would document very carefully and exactly what does this property means and
why the use should enable it or not.

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