Gnus development mailing list
 help / color / mirror / Atom feed
From: Kevin Greiner <kgreiner@xpediantsolutions.com>
Subject: Re: gnus-newsgroup-unseen gives gnus-dup-unsuppress-article error
Date: Wed, 23 Apr 2003 00:35:24 -0500	[thread overview]
Message-ID: <u3ck9dbdv.fsf@xpediantsolutions.com> (raw)
In-Reply-To: <m3lly27c18.fsf_-_@heechee.beld.net>

Ted Zlatanov <tzz@lifelogs.com> writes:

> On Tue, 22 Apr 2003, nk@viteno.net wrote:
>> Ted Zlatanov <tzz@lifelogs.com> writes:
>> 
>>> is outside spam.el, in gnus-dup.el.  I think the arrayp error is
>>> happening when gnus-dup-unsuppress-article runs
>>>
>>> (gnus-data-header (gnus-data-find article))
>>>
>>> and the article is somehow not available.  I recently switched from
>>> running through all the articles sequentially to using
>>> gnus-newsgroup-unseen and gnus-newsgroup-unreads in
>>> spam-mark-junk-as-spam-routine, could that be the problem?
>> 
>> It seems to be the problem.  I've just re-used version 6.92, ie
>> before these changes, of spam.el and everything looks great.
>> 
>> Hmm, is there a file I should, err, bring closer to reality in order
>> to use the new features?
>
> I haven't seen the error you mention.  Maybe it's Agent-related, I
> don't know, but I use the latest spam.el and I get consistently marked
> articles without errors.  I don't use the Agent.
>
> You probably have spam-mark-only-unseen-as-spam set to t (the
> default), right?  In that case the error is with
> gnus-newsgroup-unseen, it's somehow giving us articles that trigger an
> error when I try to apply the spam-mark to them (in
> gnus-dup-unsuppress-article).  Can you try to debug
> gnus-dup-unsuppress-article, using debug statements, to see where the
> arrayp error is occurring?  Since I can't duplicate your problem, you
> need to trace through it on your side.

Personally, I don't see the connection to the agent.  I've put
together this much but I don't use the spam features so this is about
all that I can do.

The macros in gnus-dup-unsupress-article expand as follows:

(defun gnus-dup-unsuppress-article (article)
  "Stop suppression of ARTICLE."
  (let ((id
          (aref
           (nth 3
                (assq article gnus-newsgroup-data))
           4)))
    ((if id
         (progn
           (setq gnus-dup-list-dirty t)
           (setq gnus-dup-list
                 (delete id gnus-dup-list))
           (unintern id gnus-dup-hashtb))))))

There's only the one aref so the error has to result from 
(assq article gnus-newsgroup-data) returning nil.  The invalid article
number that is passed to gnus-dup-unsuppress-article comes from
spam-mark-junk-as-spam-routine and, yes Ted, it gets it from
gnus-newsgroup-unseen.

The initial value of gnus-newsgroup-unseen is computed by
gnus-compute-unseen-list in gnus-sum.el.  That function computes
gnus-newsgroup-unseen as a subset of gnus-newsgroup-articles so the
initial value of gnus-newsgroup-unseen appears to be OK.

Can anyone explain how gnus-newsgroup-articles might be updated?  If
articles were removed from it, then gnus-newsgroup-unseen might become a
superset of gnus-newsgroup-articles and that would cause the arrayp error.

So far, that's all that I can identify.  I hope that this helps.

Kevin



  reply	other threads:[~2003-04-23  5:35 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-22  5:09 error in current CVS Norbert Koch
2003-04-22 13:10 ` Ted Zlatanov
2003-04-22 14:26   ` Norbert Koch
2003-04-22 16:05     ` gnus-newsgroup-unseen gives gnus-dup-unsuppress-article error (was: error in current CVS) Ted Zlatanov
2003-04-23  5:35       ` Kevin Greiner [this message]
2003-04-23 11:39         ` gnus-newsgroup-unseen gives gnus-dup-unsuppress-article error Ted Zlatanov
2003-04-23 17:50           ` Norbert Koch

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=u3ck9dbdv.fsf@xpediantsolutions.com \
    --to=kgreiner@xpediantsolutions.com \
    /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).