<p>On Oct 7, 2011 1:53 PM, "Eitan Adler" <<a href="mailto:lists@eitanadler.com">lists@eitanadler.com</a>> wrote:<br>
><br>
> On Fri, Oct 7, 2011 at 4:46 AM, Victor Engmark <<a href="mailto:victor.engmark@gmail.com">victor.engmark@gmail.com</a>> wrote:<br>
> > Hi all,<br>
> ><br>
> > I've done a bit of cleanup of the shell code:<br>
> > <<a href="https://github.com/l0b0/https-everywhere/commits/master">https://github.com/l0b0/https-everywhere/commits/master</a>><br>
><br>
> find -wholename is not portable</p>
<p>Ok, but the old code AFAICT didn't do anything useful. OTOH, Alex' changes are much more useful.</p>
<p>> +1 to the $fail changes<br>
><br>
> -[ -n "$1" ] && cd "$1"<br>
> +cd "${1:-.}"<br>
><br>
> Microoptimization?</p>
<p>It's shorter, IMO clearer, and yes, if it wasn't for those reasons, a useless microoptimization.</p>
<p>Cheers,<br>
V</p>