[Bug 3748] mass-check hangs

[bu...@bugzilla.spamassassin.org]

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=3748





--- Comment #17 from Mark Martinec <Ma...@ijs.si>  2009-01-07 18:28:35 PST ---
> Docs in Mail::SpamAssassin::Message::new says:
>   C<message> is either undef (which will use STDIN), a scalar ...
> while the actual code in Message::new treats the supplied value
> as a boolean instead of testing for defined as claimed by the docs

> Likewise, the code in Mail::SpamAssassin::Message::parse_body
> treats a user-supplied string as a boolean

Ok, fixed both in trunk:

r732591 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732591 )| mmartinec | 2009-01-08 03:17:57 +0100 (Thu, 08 Jan 2009) | 1 line
bug 3748, take 2

r732588 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732588 )| mmartinec | 2009-01-08 02:44:10 +0100 (Thu, 08 Jan 2009) | 1 line
hmm, back up the mess

r732582 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732582 )| mmartinec | 2009-01-08 02:21:41 +0100 (Thu, 08 Jan 2009) | 1 line
bug 3748: match code to docs: test $opts->{message} for defined instead of
treating it as a boolean


Sorry for the bad fix mess in r732582 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732582 )


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.