<p>Hi again,</p>
<p>Thank you for the quick reply!</p>
<p>On 16/08/06 23:37, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le lauantaina 6. elokuuta 2016, 22.31.26 EEST Filip Roséen a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Given that `RTSPClient::sendGetParameterCommand` does not accept it's third
 parameter by reference, simply passing `NULL` instead of `bye` is
 equivalent, and as such the usage of `bye` is redundant.</code></pre>
</blockquote>
<pre><code> Possibly. This unchanged from previous code.</code></pre>
</blockquote>
<p>Indeed.</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> If we do not respect the values from either of the two `rtsp->send*`, might
 this not lead to us waiting indefinitely for a reply that never happens due
 to the sending failing (for whatever reason)?</code></pre>
</blockquote>
<pre><code> I don´t know; but likewise, that´s carried over from existing code and not the 
 topic of this patch.
 </code></pre>
</blockquote>
<p>I noted on the previously mentioned since I have been told repeatedly in the past that carrying over legacy flaws when working on some implementation is not recommended (and it does not hurt hilighting potential issues if someone else feel like stepping in to fix them).</p>
<p>Though, the difference might be that it mostly happened to me when I submitted something to <code>vlc-devel</code>, and not post-commit - that distinction might be key here.</p>
<p>Best Regards,<br />
Filip Roséen</p>