https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8228
Sidney Markowitz <sidney@sidney.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |sidney@sidney.com
--- Comment #1 from Sidney Markowitz <sidney@sidney.com> ---
Actually, now that I'm reading it over, it looks like the warning has found a
longstanding bug. The documentation I linked to uses as an example exactly what
is being warned here. It says
exit $x ? 0 : 1; # actually parses as: exit($x) ? 0 : 1;
spamassassin.raw line 293 is
exit $res ? 1 : 0;
We intend exit($res ? 1 : 0);
not exit($res) ? 1 : 0;
right?
--
You are receiving this mail because:
You are the assignee for the bug.
Sidney Markowitz <sidney@sidney.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |sidney@sidney.com
--- Comment #1 from Sidney Markowitz <sidney@sidney.com> ---
Actually, now that I'm reading it over, it looks like the warning has found a
longstanding bug. The documentation I linked to uses as an example exactly what
is being warned here. It says
exit $x ? 0 : 1; # actually parses as: exit($x) ? 0 : 1;
spamassassin.raw line 293 is
exit $res ? 1 : 0;
We intend exit($res ? 1 : 0);
not exit($res) ? 1 : 0;
right?
--
You are receiving this mail because:
You are the assignee for the bug.