<html><head></head><body>Hi,<br><br>I don't think that there is a question whether there should always be a pool, because it now depends on the decoder plugin. We don't know what the decoder does or does not do internally, so there are no ways to generically allocate a pool outside on a systematic basis.<br><br>And it would be insane to allocate a whole pool for image decoders, especially considering that pictures tend to be larger than video frames. Image decoders should allocate their picture explicitly.<br><br>No, the first question is what would the determinant be to allocate a pool for the decoder or let the decoder manage its pool.<br><br><div class="gmail_quote">Le 20 février 2019 12:01:52 GMT+02:00, Steve Lhomme <robux4@ycbcr.xyz> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">On 20/02/2019 09:56, Thomas Guillem wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Please,  send future decoder patches on the ML first.<br></blockquote><br>BTW do we agree on my 13+1 patchset, generalizing the work on the output <br>format passed to update_format() ?<br><br>Later this format will be the one used for the decoder pool. And <br>possibly set on dec->fmt_out.video, although in my branch I kept a <br>separate one to check the pool format against it.<hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>