all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eric Abrahamsen <eric@ericabrahamsen.net>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Nikolaus Rath <Nikolaus@rath.org>, emacs-devel@gnu.org
Subject: Re: How to help with bug 21071
Date: Sat, 31 Dec 2016 11:04:47 -0800	[thread overview]
Message-ID: <878tqvc434.fsf@ericabrahamsen.net> (raw)
In-Reply-To: <837f6ivijl.fsf@gnu.org> (Eli Zaretskii's message of "Thu, 29 Dec 2016 17:53:02 +0200")

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Nikolaus Rath <Nikolaus@rath.org>
>> Date: Wed, 28 Dec 2016 16:35:46 -0800
>> 
>> What is the best way for me to fix bug
>> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=21071. This is about IMAP
>> mails not being marked as read when they are expired at the same time.
>> 
>> I'm happy to write a patch, but it's not clear to me what the patch
>> should do. Is the suggestion in message #11 a good idea?
>
> Could some Gnus maintainer please answer Nikolaus's questions?
> Thanks.

I'm by no means a Gnus maintainer, but I'm somewhat familiar with the
code, and would opt for Nikolaus' second suggestion on the bug tracker:
taking `gnus-summary-expire-articles' out of
`gnus-summary-prepare-exit-hook', and explicitly calling it in
`gnus-summary-exit'.

My understanding of hooks is they're meant more for user customization,
but running the expiration is Gnus plumbing (if users don't want to
expire articles, they already have plenty of ways of doing that). And it
seems evident that setting marks should happen before expiration (at
least to me!).

Eric



      reply	other threads:[~2016-12-31 19:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-29  0:35 How to help with bug 21071 Nikolaus Rath
2016-12-29 15:53 ` Eli Zaretskii
2016-12-31 19:04   ` Eric Abrahamsen [this message]

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=878tqvc434.fsf@ericabrahamsen.net \
    --to=eric@ericabrahamsen.net \
    --cc=Nikolaus@rath.org \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.