Gnus development mailing list
 help / color / mirror / Atom feed
From: Ted Zlatanov <tzz@lifelogs.com>
To: ding@gnus.org
Subject: spam2.el again (was: bug#9434: [nnir] error quitting group)
Date: Thu, 26 Jan 2012 16:45:59 -0600	[thread overview]
Message-ID: <8739b25bh4.fsf_-_@lifelogs.com> (raw)
In-Reply-To: <87y5swkbz8.fsf@lifelogs.com>

Following up with a changed subject because the original subject was
completely wrong.

Ted

On Tue, 24 Jan 2012 15:49:15 -0600 Ted Zlatanov <tzz@lifelogs.com> wrote: 

TZ> On Sun, 04 Sep 2011 18:44:08 -0800 Dave Abrahams <dave@boostpro.com> wrote: 
>>> This is definitely NOT the case for nnir right now (but will be as
>>> soon as I push my re-write). So I might be wrong about the
>>> spam-related variables being global, and I'm sorry I suggested
>>> it. Maybe Ted can set me straight?

DA> I sure hope it's supported.

TZ> The spam.el code is not buffer-localized.  All the variables it creates
TZ> are global.  I'd rather rewrite the whole thing as spam2.el than try to
TZ> fix this one thing, because there are so many other issues with spam.el:

TZ> - hard to configure

TZ> - not loaded by default

TZ> - not well integrated with the Gnus registry

TZ> - old and unused spam/ham processors

TZ> - copy/move logic needs to be much cleaner

TZ> - general logic is hard to follow, I was learning ELisp (I still am, but
TZ>   hey, it's only been 15 years)

TZ> - elegant original design by Francois Pinard corrupted by my primitive
TZ>   knowledge of ELisp ;)

TZ> - many new ideas floating in my head, especially "jam" handling (spam
TZ>   you have legitimately requested)

TZ> - at least 2 old threads had improvement suggestions for spam2.el, and
TZ>   I've saved them for a rainy day

TZ> CC to ding mailing list for comments.




  reply	other threads:[~2012-01-26 22:45 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <m2fwkc7s2x.fsf@boostpro.com>
     [not found] ` <8762l7ripv.fsf@andy.bu.edu>
     [not found]   ` <m2hb4r3iqf.fsf@boostpro.com>
     [not found]     ` <87y5y3pzjv.fsf@andy.bu.edu>
     [not found]       ` <87obyzpy3s.fsf@andy.bu.edu>
     [not found]         ` <m2k49n1zev.fsf@boostpro.com>
2012-01-24 21:49           ` bug#9434: [nnir] error quitting group Ted Zlatanov
2012-01-26 22:45             ` Ted Zlatanov [this message]
2012-12-26 17:46             ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8739b25bh4.fsf_-_@lifelogs.com \
    --to=tzz@lifelogs.com \
    --cc=ding@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).