<div><div dir="auto">Alright, thanks for looking into this Thomas :-)</div></div><div dir="auto"><br></div><div dir="auto">Roland</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 7 May 2019 at 10:25 PM, Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Actually, this is not perfect either. cf. <a href="https://trac.videolan.org/vlc/ticket/22273#ticket" rel="noreferrer" target="_blank">https://trac.videolan.org/vlc/ticket/22273#ticket</a><br>
<br>
But I won't ask you to fix this issue. I will take care of it. In the meantime, we can stay with the vout_AddSubpictureClock/vout_RemoveSubpictureClock solution.<br>
<br>
On Tue, May 7, 2019, at 06:57, Roland Bewick wrote:<br>
> <br>
> On 6/05/2019 2:36 PM, Thomas Guillem wrote:<br>
> > Hello,<br>
> ><br>
> > On Sun, May 5, 2019, at 12:03, Roland Bewick wrote:<br>
> >> ---<br>
> >>   src/input/decoder.c                 | 15 ++++++---<br>
> >>   src/video_output/video_output.c     | 17 ++++++----<br>
> >>   src/video_output/vout_internal.h    | 14 ++++----<br>
> >>   src/video_output/vout_subpictures.c | 50 +++++++++++++++++------------<br>
> >>   4 files changed, 59 insertions(+), 37 deletions(-)<br>
> >><br>
> >> diff --git a/src/input/decoder.c b/src/input/decoder.c<br>
> >> index b5fe1c2ca3..b699e52a67 100644<br>
> >> --- a/src/input/decoder.c<br>
> >> +++ b/src/input/decoder.c<br>
> >> @@ -612,7 +612,8 @@ static subpicture_t *spu_new_buffer( decoder_t *p_dec,<br>
> >>           if( p_owner->p_vout )<br>
> >>           {<br>
> >>               vlc_mutex_lock( &p_owner->lock );<br>
> >> -            vout_SetSubpictureClock(p_owner->p_vout, NULL);<br>
> >> +            vout_SetSubpictureClock(p_owner->p_vout, NULL,<br>
> >> +                                    p_owner->b_secondary);<br>
> > I would prefer something like<br>
> >   int vout_AddSubpictureClock(vlc_clock_t *);<br>
> >   void vout_RemoveSubpictureClock(vlc_clock_t *),<br>
> ><br>
> > We won't have to change if we need to handle 3 SPUS. The vout can just keep an array of 2 clocks and return an error if it receive a third vout_AddSubpictureClock().<br>
> <br>
> Nice idea - will do.<br>
> <br>
> Thanks,<br>
> <br>
> Roland<br>
> <br>
><br>
</blockquote></div></div>