[Bug 775490] vaapi: encode: jpeg should release configuration after each frame (multi file support)

[GStreamer (GNOME Bugzilla)]

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

--- Comment #26 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
(In reply to Hyunjun Ko from comment #25)
> (In reply to Víctor Manuel Jáquez Leal from comment #24)
> > Summarizing, right now in master is possible to encode multiple images of
> > the same caps. But, not for different ones, is that case still the matter of
> > this bug? Or do we close it now?
> 
> In Stirling's patch, the following is worth merge, I guess.
> Looks it has potential bugs without this, even though it doesn't affect at
> the moment.
> 
> encoder->num_codedbuf_queued--;

That cannot be merged because that is not the intention of that variable AFAIK.
It is a global counter of processed buffers, not the size of the queue.

Though, it seems the fix should be in gst_vaapi_encoder_set_codec_state()

> 
> 
> And what about this?
> call gst_vaapiencode_flush (venc); in stop method.

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