Probable misuse of "" instead of "or".

I haven't bothered to read your code in depth but you are probably misusing the "||" operator. You should be using "or" and your open statements should look like this:

open (SCEMAIL, "<$spamcop_email") or die "Can't open $spamcop_email";

and not this:

open (SCEMAIL, "$spamcop_email") || die "Can't open $spamcop_email";

Note: I only threw in the "<" char in the open statement because I like completeness although it is not necessary.

Your problems with the output may or may not be due to this problem.

metadoktor

"The doktor is in."

Comment on Probable misuse of "" instead of "or".

Replies are listed 'Best First'.
Re: Probable misuse of "" instead of "or". by IlyaM (Parson) on Jan 05, 2002 at 19:02 UTC
As long as open have its parameters in brackets `\|\|` is ok. However I do prefer `or` version since it allows me to use open without brackets. That is: `open SCEMAIL, "< $spamcop_email" or die "Can't open $spamcop_email: $! +";` [download] Note: It is good idea to add `$!` in string printed by die because it helps to diagnose the problem which caused fail of open. -- Ilya Martynov (http://martynov.org/)	[reply] [d/l] [select]

Replies are listed 'Best First'.

Re: Probable misuse of "" instead of "or".
by IlyaM (Parson) on Jan 05, 2002 at 19:02 UTC

open

||

or

open


open SCEMAIL, "< $spamcop_email" or die "Can't open $spamcop_email: $!
+";
[download]

Note:

$!

die

open

--
Ilya Martynov (http://martynov.org/)

[reply]
[d/l]
[select]