Gnus development mailing list
 help / color / mirror / Atom feed
From: Simon Josefsson <jas@extundo.com>
Cc: ding@gnus.org
Subject: Re: IMAP article move optimization!
Date: Tue, 01 Mar 2005 17:51:26 +0100	[thread overview]
Message-ID: <iluzmxnpawh.fsf@latte.josefsson.org> (raw)
In-Reply-To: <4nsm3fpc1a.fsf@lifelogs.com> (Ted Zlatanov's message of "1 Mar 2005 11:26:57 -0500")

"Ted Zlatanov" <tzz@lifelogs.com> writes:

> On Tue, 01 Mar 2005, yamaoka@jpl.org wrote:
>
>>>>>> In <4nhdjwg1nn.fsf@lifelogs.com> Ted Zlatanov wrote:
>> 
>>>> Isn't it better to use defvar to gnus-sum-hint-move-is-internal?
>> 
>>> I wanted it to be a local variable without definition or effect
>>> outside its scope.  It's a kludge anyhow, the real solution is to have
>>> nnimap-request-move-article check the source and destination server.
>>> Do you agree?
>> 
>> I see.  Sorry to have written rubbish.
>
> Your point was valid!  If it was rubbish I would have told you I
> disagree :)
>
> I am just asking you and Simon if you agree that
> nnimap-request-move-article should check the servers for equality.  I
> can also put that as a parameter to gnus-request-move-article and have
> it be passed to ALL move commands - nnmaildir, nnml, etc.  Is that
> better?  Do other backends benefit?  I would imagine nnml and nnfolder
> won't, but maybe nnmaildir would benefit.

Would this mean changing the back end interface?  I'm not sure it is
clean enough to warrant that, but I haven't really thought about it.

I haven't really read your patches, but why would n-r-m-a have to
check server equality if g-s-h-m-i-i is non-nil?

Btw, at least internal move appear to work here, although it flashes
something about "internal move" in the echo area.  Moving several
messages appear to be possible to optimize further, though, since it
select the new and old mailbox for each flag set, and the flags could
be set finally.  But let's not think about that more until this
approach has settled down.

Perhaps I'm not of much help here, I wish I had more time to tinker
with this.  Feel free to install anything you think is good.



  reply	other threads:[~2005-03-01 16:51 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-08 19:19 IMAP article move Ted Zlatanov
2004-11-08 20:50 ` Raymond Scholz
2004-11-08 21:15 ` Simon Josefsson
2004-11-09 22:00   ` Ted Zlatanov
2004-11-09 22:06     ` Simon Josefsson
2005-02-25 19:59       ` IMAP article move optimization! (was: IMAP article move) Ted Zlatanov
2005-02-28  0:00         ` IMAP article move optimization! Katsumi Yamaoka
2005-02-28 15:11           ` Ted Zlatanov
2005-03-01  0:37             ` Katsumi Yamaoka
2005-03-01 16:26               ` Ted Zlatanov
2005-03-01 16:51                 ` Simon Josefsson [this message]
2005-03-03 18:04                   ` Ted Zlatanov
2005-03-02  0:39                 ` Katsumi Yamaoka
2005-01-10 19:41   ` IMAP article move David Abrahams
2005-02-25 20:04     ` Ted Zlatanov
2005-03-03 18:05       ` Ted Zlatanov
2005-03-03 19:09         ` David Abrahams
2005-03-04 16:10           ` Ted Zlatanov
2005-03-21  9:32         ` David Abrahams
2005-03-22 19:14           ` Ted Zlatanov
2005-03-22 22:09             ` Simon Josefsson
2005-03-23 19:46               ` Ted Zlatanov
2005-01-11  9:16   ` Bjørn Mork

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=iluzmxnpawh.fsf@latte.josefsson.org \
    --to=jas@extundo.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).