From: Richard Stallman <rms@gnu.org>
To: Chong Yidong <cyd@stupidchicken.com>
Cc: eliz@gnu.org, emacs-devel@gnu.org
Subject: Re: Upcoming 23.1 release
Date: Tue, 21 Jul 2009 10:41:52 -0400 [thread overview]
Message-ID: <E1MTGXQ-0006S4-AC@fencepost.gnu.org> (raw)
In-Reply-To: <874ot787su.fsf@stupidchicken.com> (message from Chong Yidong on Mon, 20 Jul 2009 15:37:37 -0400)
Secondly, because of Rmail-mbox's funky
buffer-swapping design, it's not even clear to me that turning on
auto-save-mode manually in Rmail leads to the correct buffer being
auto-saved.
The same hook that does it for real saves
runs in this case too.
I just tested it again. It took just 30 seconds to do M-x
auto-save-mode followed by running (do-auto-save). However, I kept
editing and I saw that it kept saving the whole file.
I just found and fixed a bug in my change: auto-save-mode did not
toggle the mode off in an Rmail file. But even if that had not been
fixed, it is less severe than the bug now planned for 23.1.
I think the correct fix is to change `buffer-swap-text', giving it an
optional argument that makes it swap the two buffers' values of
auto-save-file-name and auto_save_modified (and possibly other
file-related variables also). And we need an additional hook into
auto-save-mode that Rmail can use to ensure that M-x auto-save-mode
turns on auto-save in the correct buffer.
That approach might work too. But we have a fix that works, and is
much simpler. It is better to use this, and fix the bug.
next prev parent reply other threads:[~2009-07-21 14:41 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-20 18:32 Upcoming 23.1 release Chong Yidong
2009-07-20 18:44 ` Eli Zaretskii
2009-07-20 18:51 ` Chong Yidong
2009-07-20 19:06 ` Eli Zaretskii
2009-07-20 19:37 ` Chong Yidong
2009-07-20 20:36 ` Eli Zaretskii
2009-07-21 1:20 ` Stefan Monnier
2009-07-21 15:05 ` Chong Yidong
2009-07-21 15:42 ` Stefan Monnier
2009-07-21 14:41 ` Richard Stallman [this message]
2009-07-21 14:41 ` Richard Stallman
2009-07-21 15:38 ` Chong Yidong
2009-07-22 1:43 ` Richard Stallman
2009-07-20 19:02 ` Chong Yidong
2009-07-20 20:42 ` Eli Zaretskii
2009-07-21 15:31 ` Chong Yidong
2009-07-22 1:43 ` Richard Stallman
2009-07-21 14:42 ` Richard Stallman
2009-07-21 14:41 ` Richard Stallman
2009-07-20 18:55 ` Dan Nicolaescu
2009-07-20 19:06 ` Chong Yidong
2009-07-21 3:16 ` usefulness of no-byte-compile [was: Re: Upcoming 23.1 release] Dan Nicolaescu
2009-07-20 20:05 ` Upcoming 23.1 release Chong Yidong
2009-07-20 22:58 ` Jason Rumney
2009-07-20 23:12 ` Chong Yidong
2009-07-21 0:43 ` Kenichi Handa
2009-07-21 3:10 ` Eli Zaretskii
2009-07-21 3:55 ` Lennart Borgman
2009-07-22 6:55 ` Yavor Doganov
2009-07-22 7:24 ` David Reitter
2009-07-24 0:46 ` YAMAMOTO Mitsuharu
2009-07-24 0:53 ` YAMAMOTO Mitsuharu
2009-07-24 15:10 ` Yavor Doganov
2009-07-24 15:47 ` Adrian Robert
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=E1MTGXQ-0006S4-AC@fencepost.gnu.org \
--to=rms@gnu.org \
--cc=cyd@stupidchicken.com \
--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.