You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2014/01/08 16:34:08 UTC
[Bug 6990] Redis.pm fails perlcritic test for package name
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6990
Mark Martinec <Ma...@ijs.si> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|3.4.0 |3.4.1
Severity|normal |enhancement
--- Comment #1 from Mark Martinec <Ma...@ijs.si> ---
Yes, I'm aware of it, Kevin pointed it out on 2013-10-11.
I prefer not to touch this before the 3.4.0 release. It's just a
'good programming practice issue', otherwise it's nothing wrong
with changing namespaces within a single perl module.
Eventually the Redis module could make it into a separate file
under Util/ when there is a need to use it for some other task
besides the bayes database. Now that we are in the 3.4.0-RC phase
this is not the best time for such reorganization - it will have
to wait till next release IMO.
--
You are receiving this mail because:
You are the assignee for the bug.