Re: shift or die unless

Personally, I think you're trying to do too much in that one line. It's not immediately clear when I look at it whether it means

($arg = shift || die "takes one arg") unless -M $file >7;
[download]

$arg = shift || (die "takes one arg" unless -M $file >7;)
[download]

It would, IMO, be much better written as two separate statements.

Comment on Re: shift or die unless Select or Download Code

Replies are listed 'Best First'.
Re^2: shift or die unless by ikegami (Patriarch) on Dec 03, 2008 at 18:38 UTC
Only if you think `unless` is an operator? The second won't even compile.	[reply] [d/l]
Re^3: shift or die unless by dsheroh (Monsignor) on Dec 03, 2008 at 22:30 UTC
It wasn't meant to be runnable code, just to indicate logical grouping. More verbosely: It is not immediately obvious whether that line means "check the date of the file and, if it's less than a week old, shift or die" or "if shift returns a false value and the file is less than a week old, then die".	[reply]
Re^4: shift or die unless by ikegami (Patriarch) on Dec 04, 2008 at 12:16 UTC
If it's not runnable code, the logical grouping is impossible.	[reply]
Re^5: shift or die unless by eric256 (Parson) on Dec 04, 2008 at 15:28 UTC
Re^6: shift or die unless by ikegami (Patriarch) on Jan 09, 2009 at 17:10 UTC