On Aug 22, 12:50 am, [EMAIL PROTECTED] (Yitzle) wrote:
> http://perldoc.perl.org/functions/grep.html

Good advice.  Irrelevant to the problem at hand, but good advice
nonetheless.

> Without testing the code or anything, could it be that:
> unless (grep $item eq $_, %$all_ref{$crew}) {
>
> should be:
> unless (grep $item eq %$all_ref{$crew}) {

Bad advice.  Very very bad advice.  That makes *no* sense of any
kind.  Even if %$all_ref{$crew} were a valid dereference, what would
make you think this suggestion was valid?  Did you read the link you
posted?   Even if you didn't, how does comparing a scalar to a hash
make any sense?  How does eliminating the LIST passed to grep() make
any sense?

I'm sure you were trying to help, which is respectable, but really
giving bad advice and throwing things at the wall hoping something
sticks is far far worse than saying silent.

Paul Lalli


-- 
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
http://learn.perl.org/


Reply via email to