http://bugzilla.spamassassin.org/show_bug.cgi?id=3701
Summary: clean up debugging
Product: Spamassassin
Version: SVN Trunk (Latest Devel Version)
Platform: Other
OS/Version: other
Status: NEW
Severity: normal
Priority: P5
Component: Libraries
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
Debugging is a bit of a mess. It's very all or none and changing debugging
levels for different sections of code is a real pain. For 3.1, I propose
a minimal set of changes to fix the major problems.
First, we will continue to have three "levels" of importance: die(), warn(),
and dbg(). Later, we can consider an additional info() level between warn()
and dbg(). Later.
At all times, die() and warn() messages are output. dbg() messages remain
optional.
As much as possible, all die(), warn(), and dbg() messages will be prefixed
with a facility name, like "dns: foo", "bayes: bar", or "eval: gurgle".
-D means debugging is on for everything
-D=dns,bayes means debugging is turned on for those facilities
The facilties debugging list is just an array stored somewhere.
Then dbg("string") is only printed if $1 from /^([^:]+)/ is listed in that
array (or "all") is set. The facility names are purely informational and for
consistency in warn() and die() messages.
So you'd better use a facility unless you only want your dbg() be printed for
"all".
And goodbye to the integers! Goodbye... goodbye.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
