Gnus development mailing list
 help / color / mirror / Atom feed
From: Wolfgang Jenkner <wjenkner@inode.at>
To: Matt Ford <matt@dancingfrog.co.uk>
Cc: ding@gnus.org
Subject: Re: `C-u SPC' vs `/ o' odd behaviour
Date: Wed, 11 Apr 2012 20:24:46 +0200	[thread overview]
Message-ID: <857gxmdt3l.fsf@iznogoud.viz> (raw)
In-Reply-To: <877gxm36al.fsf@dancingfrog.co.uk> (Matt Ford's message of "Wed, 11 Apr 2012 11:35:14 +0100")

On Wed, Apr 11 2012, Matt Ford wrote:

> Essentially with GNUs agent switched on both of my configured IMAP
> servers (one google, one exchange) then the behaviour of `/ o' when in
> the summary buffer is buggy.
>
> When in a summary buffer `/ o' reports the maximum UID in the prompt
> rather than then correct number of articles and does not return the
> requested amount of articles - typically I get far less.

I have the following preliminary patch to fix A T with the agent (and
the nntp backend).  Could you try if it fixes your problem, too ?

Wolfgang

From 73a7ab983ec5a850d2f7b786e074d7bbd2fe0bdf Mon Sep 17 00:00:00 2001
From: Wolfgang Jenkner <wjenkner@inode.at>
Date: Thu, 23 Feb 2012 03:19:00 +0100
Subject: [PATCH] When the agent is active gnus-summary-refer-thread inserts
 only headers which are already in the agent cache but it
 doesn't try to fetch old headers.

Check this fix: Using uncached-articles or (car articles) might not be
quite right (but then again, it might).
---
 lisp/gnus-agent.el |    7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/lisp/gnus-agent.el b/lisp/gnus-agent.el
index 540a97b..f8baaf1 100644
--- a/lisp/gnus-agent.el
+++ b/lisp/gnus-agent.el
@@ -3743,6 +3743,13 @@ has been fetched."
       (gnus-make-directory (nnheader-translate-file-chars
 			    (file-name-directory file) t))
 
+      (when fetch-old
+	(setq articles (gnus-uncompress-range
+			(cons (if (numberp fetch-old)
+				  (max 1 (- (car articles) fetch-old))
+				1)
+			      (car (last articles))))))
+
       ;; Populate temp buffer with known headers
       (when (file-exists-p file)
 	(with-current-buffer gnus-agent-overview-buffer
-- 
1.7.9.6





  reply	other threads:[~2012-04-11 18:24 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-11 10:35 Matt Ford
2012-04-11 18:24 ` Wolfgang Jenkner [this message]
2012-04-12 10:53   ` Matt Ford
2012-06-10 20:51 ` Lars Magne 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=857gxmdt3l.fsf@iznogoud.viz \
    --to=wjenkner@inode.at \
    --cc=ding@gnus.org \
    --cc=matt@dancingfrog.co.uk \
    /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).