Julien Danjou | 15 Jun 2012 11:52
Gravatar

[gnus git] branch master updated: m0-5-39-g4c4bb35 =1= nnimap: possibly-change-group get a read-only argument

       via  4c4bb3503d9141fa298eb5ba29d3012669f7f73a (commit)
      from  7a78109abe930702366ca2978d804779504d5335 (commit)

- Log -----------------------------------------------------------------
commit 4c4bb3503d9141fa298eb5ba29d3012669f7f73a
Author: Julien Danjou <julien <at> danjou.info>
Date:   Fri Jun 15 11:49:20 2012 +0200

    nnimap: possibly-change-group get a read-only argument

    Signed-off-by: Julien Danjou <julien <at> danjou.info>

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index c548b7e..6a1cd0c 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
 <at>  <at>  -2,6 +2,11  <at>  <at> 

 	* nnimap.el (nnimap-find-article-by-message-id): Use
 	`nnimap-possibly-change-group' rather than its own EXAMINE call.
+	(nnimap-possibly-change-group): Add read-only argument.
+	(nnimap-request-list): Use nnimap-possibly-change-group rather than
+	issuing EXAMINE manually.
+	(nnimap-find-article-by-message-id): Use `nnimap-possibly-change-group'
+	with read-only argument.

 2012-06-11  Lars Magne Ingebrigtsen  <larsi <at> gnus.org>

diff --git a/lisp/nnimap.el b/lisp/nnimap.el
index 211b8c8..a911f52 100644
(Continue reading)

Lars Ingebrigtsen | 4 Sep 2012 20:44
Picon
Favicon
Gravatar

Re: [gnus git] branch master updated: m0-5-39-g4c4bb35 =1= nnimap: possibly-change-group get a read-only argument

Julien Danjou <julien <at> danjou.info> writes:

> +	(nnimap-request-list): Use nnimap-possibly-change-group rather than
> +	issuing EXAMINE manually.

Julien, was there a particular reason for doing this?
`nnimap-request-list' used to stream the EXAMINE commands, while the new
version is synchronous, which is s-l-o-w on lines with higher latencies.

If this didn't fix any particular bug, can you revert the patch?

--

-- 
(domestic pets only, the antidote for overdose, milk.)
  http://lars.ingebrigtsen.no  *  Sent from my Emacs

Julien Danjou | 5 Sep 2012 11:27
Gravatar

Re: [gnus git] branch master updated: m0-5-39-g4c4bb35 =1= nnimap: possibly-change-group get a read-only argument

On Tue, Sep 04 2012, Lars Ingebrigtsen wrote:

> Julien, was there a particular reason for doing this?

Not, it was just to do code cleanup/factorisation. Ya know.

> If this didn't fix any particular bug, can you revert the patch?

Yup.

--

-- 
Julien Danjou
# Free Software hacker & freelance
# http://julien.danjou.info

Gmane