<div dir="ltr"><div>Thanks! How can I update my patch to remove the concerned lines?</div><div><br></div><div>Regards,</div><div>-- <br></div><div>Yohann D'ANELLO<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 28 oct. 2020 à 11:53, Alexandre Janniaux <<a href="mailto:ajanni@videolabs.io">ajanni@videolabs.io</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Oct 28, 2020 at 11:51:58AM +0100, Ÿnérant wrote:<br>
> Hi,<br>
><br>
> Le mer. 28 oct. 2020 à 10:35, Marvin Scholz <<a href="mailto:epirat07@gmail.com" target="_blank">epirat07@gmail.com</a>> a écrit :<br>
><br>
> > Hi,<br>
> ><br>
> > On 28 Oct 2020, at 0:57, Ÿnérant wrote:<br>
> ><br>
> > > diff --git a/modules/access/srt.c b/modules/access/srt.c<br>
> > > index 499a300196..ae8ba2a84f 100644<br>
> > > --- a/modules/access/srt.c<br>
> > > +++ b/modules/access/srt.c<br>
> > > @@ -97,6 +97,8 @@ static bool srt_schedule_reconnect(stream_t<br>
> > > *p_stream)<br>
> > >      int stat;<br>
> > >      char *psz_passphrase = var_InheritString( p_stream,<br>
> > > SRT_PARAM_PASSPHRASE );<br>
> > >      bool passphrase_needs_free = true;<br>
> > > +    char *psz_streamid = var_InheritString( p_stream,<br>
> > > SRT_PARAM_STREAMID );<br>
> > > +    bool streamid_needs_free = true;<br>
> ><br>
> > Instead of a separate bool for this, maybe just always strdup?<br>
><br>
><br>
> I copied what was written for the passphrase, I don't know if we could<br>
> write better. I am not a C programmer, so I don't know how to optimize<br>
> memory, I'm sorry.<br>
><br>
> ><br>
> > >      char *url = NULL;<br>
> > >      srt_params_t params;<br>
> > >      struct addrinfo hints = {<br>
> > > @@ -145,6 +147,11 @@ static bool srt_schedule_reconnect(stream_t<br>
> > > *p_stream)<br>
> > >                  passphrase_needs_free = false;<br>
> > >                  psz_passphrase = (char *) params.passphrase;<br>
> > >              }<br>
> > > +    if (params.streamid != NULL ) {<br>
> > > + free( psz_streamid );<br>
> > > + streamid_needs_free = false;<br>
> > > + psz_streamid = (char *) params.streamid;<br>
> > > +    }<br>
> > >          }<br>
> > >      }<br>
> > ><br>
> > > @@ -164,7 +171,7 @@ static bool srt_schedule_reconnect(stream_t<br>
> > > *p_stream)<br>
> > ><br>
> > >      /* Set latency */<br>
> > >      srt_set_socket_option( strm_obj, SRT_PARAM_LATENCY, p_sys->sock,<br>
> > > -            SRTO_TSBPDDELAY, &i_latency, sizeof(i_latency) );<br>
> > > +            SRTO_LATENCY, &i_latency, sizeof(i_latency) );<br>
> ><br>
> > This change seems unrelated to this patch?<br>
><br>
><br>
> Hum yeah, but without this change, the code is not compiling, since it uses<br>
> deprecated SRT api. I left it since it is only one line...<br>
<br>
I'll merge the already submitted patch fixing that right now. ;)<br>
<br>
Regards,<br>
--<br>
Alexandre Janniaux<br>
Videolabs<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>