On 19 Jan 2004, Randal L. Schwartz wrote:

> >>>>> "Simon" == Simon Wilcox <[EMAIL PROTECTED]> writes:
> 
> Simon> -                    mkdir $target, $mode || do {
> Simon> -                        warn "mkdir  ($target): $!\n";
> Simon> +                    mkpath $target, $mode || do {
> Simon> +                        warn "mkpath  ($target): $!\n";
> 
> I question either version of this code.  The || there
> is higher precedence than the list-building of mkdir or mkpath,
> so you're essentially coding:
> 
>         mkdir $target, ($mode || ... );
> 
> Is that intended?  I suspect not.

Good point. I can't say I was exactly looking too hard at it (abw's code 
usually Just Works [tm] in my experience :).

Sadly I don't have time right now to code the right fix.

"Patches welcome" ;-)

S.


_______________________________________________
templates mailing list
[EMAIL PROTECTED]
http://lists.template-toolkit.org/mailman/listinfo/templates

Reply via email to