<div dir="ltr"><div dir="ltr">On Tue, Mar 10, 2020 at 9:55 PM Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le sunnuntaina 8. maaliskuuta 2020, 19.14.41 EET Kartik Ohri a écrit :<br>
> +    i_ret = vlc_tls_Read (sock, p_buffer, sizeof (p_buffer) - 1, false);<br>
> +    msg_Dbg (p_this, "Response: %s", (char *) p_buffer);<br>
> +    vlc_tls_Close (sock);<br>
> +    if ( i_ret <= 0 )<br>
> +    {<br>
> +        msg_Warn (p_this, "No response");<br>
> +        return 1;<br>
> +    }<br>
> +    p_buffer[i_ret] = '\0';<br>
> +    if ( strstr ((char *) p_buffer, "OK") )<br>
<br>
This is very sloppy coding.<br></blockquote><div>I do not have much experience on working on writing something ground up or without guidance in a real world project. Can you point out the issue ? I will fix that in the next patch.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +    {<br>
> +        vlc_vector_clear (&p_sys->p_queue);<br>
> +        msg_Dbg (p_this, "Submission successful!");<br>
> +    }<br>
> +    else<br>
> +    {<br>
> +        msg_Warn (p_this, "Error: %s", (char *) p_buffer);<br>
> +        return 1;<br>
> +    }<br>
> +<br>
> +    return 0;<br>
> +}<br>
<br>
-- <br>
レミ・デニ-クールモン<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
<br>
<br>
<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></div>