<html><head></head><body>It's still a bug to update the video format of a non-video ES. Patch is excessively broad IMO<br><br><div class="gmail_quote">Le 28 octobre 2019 14:41:25 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">Many of them do trivial things that can be factorized in decoder_helpers.<hr> src/input/decoder_helpers.c | 6 ++++--<br> 1 file changed, 4 insertions(+), 2 deletions(-)<br><br>diff --git a/src/input/decoder_helpers.c b/src/input/decoder_helpers.c<br>index ce32608c92c..87ab6bc560f 100644<br>--- a/src/input/decoder_helpers.c<br>+++ b/src/input/decoder_helpers.c<br>@@ -84,13 +84,15 @@ int decoder_UpdateVideoFormat( decoder_t *dec )<br> int decoder_UpdateVideoOutput( decoder_t *dec, vlc_video_context *vctx_out )<br> {<br> vlc_assert( dec->fmt_in.i_cat == VIDEO_ES && dec->cbs != NULL );<br>- if ( unlikely(dec->fmt_in.i_cat != VIDEO_ES || dec->cbs == NULL ||<br>- dec->cbs->video.format_update == NULL) )<br>+ if ( unlikely(dec->fmt_in.i_cat != VIDEO_ES || dec->cbs == NULL) )<br> return -1;<br> <br> /* */<br> dec->fmt_out.video.i_chroma = dec->fmt_out.i_codec;<br> <br>+ if (dec->cbs->video.format_update == NULL)<br>+ return 0;<br>+<br> return dec->cbs->video.format_update( dec, vctx_out );<br> }<br> </pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>