<div dir="ltr">On Sat, Mar 2, 2013 at 1:46 PM, Michal Sekletar <span dir="ltr"><<a href="mailto:sekletar.m@gmail.com" target="_blank">sekletar.m@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">>> -                        if (!i->type == IGNORE_DIRECTORY_PATH || !streq(dent->d_name, p)) {<br>


>> +                        if (!(i->type == IGNORE_DIRECTORY_PATH || !streq(dent->d_name, p))) {<br>
> After the change we have:<br>
>  if (i->type != I_D_P && streq(d_name, p))<br>
<br>
</div>Agreed.<br>
<div class="im">><br>
> Shouldn't we instead have<br>
>  if (i->type != I_D_P || strneq(d_name, p))<br>
> ?<br>
><br>
<br>
</div>I don't think so, because we want to get rid of directory, when item type is different from IGNORE_DIRECTORY_PATH or if we are processing item of type I_D_N, but directory names does not match. I *assume* that original behaviour is actually correct.<br>

</blockquote><div><br></div><div style>Thing is, you didn't kept original behaviour ;)</div><div style><br></div><div style>(!a || !b) == !(a && b) != !(a || !b)</div><div style><br></div><div style>or am I missing something here? ;)</div>

</div><br></div></div>