<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 21, 2014 at 11:11 AM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Laurent established the convention of keeping the existing style of existing<br>
files. I don't mind removing the extra whitespace between function identifier<br>
and parenthesis, but the rest is pretty much standard.<br></blockquote><div><br></div><div>Any non-respect of this rule from me must be treated as a typo. I usually try to pay attention to this, but i've rewritten this piece of code so many times now  so i guess some have slipped through my attention.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">What was the problem? Reordering code for no reasons make it really hard to<br>
review.<br>
<div><div class="h5"><br></div></div></blockquote><div><br></div><div>Please accept all my apologies for this, i've been taking bits and pieces from the different previous patches and it resulted in that. I'll fix this</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><br></div></div><div><div class="h5">
>  error:<br>
> -    closedir (handle);<br>
> -    free (p_sys);<br>
> +    closedir( handle );<br>
> +    if( p_sys != NULL )<br>
<br>
</div></div>Pointless.<br></blockquote><div><br></div><div>(facepalm)</div><div> </div><div><span style="color:rgb(80,0,80)">> +        //msg_Dbg( p_access, "|||||| psz_full_uri = %s ", psz_full_uri );</span></div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
<br>
</div></div>???<br></blockquote><div><br></div><div>(double facepalm) </div><div><br></div></div></div></div>