GNU bug report logs - #2137
23.0.60; Saving Rmail buffer does not show the "Saving file ..." message

Previous Next

Package: emacs;

Reported by: Eli Zaretskii <eliz <at> gnu.org>

Date: Sat, 31 Jan 2009 19:25:04 UTC

Severity: normal

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 2137 in the body.
You can then email your comments to 2137 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs. (Sat, 31 Jan 2009 19:25:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
New bug report received and forwarded. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>. (Sat, 31 Jan 2009 19:25:05 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: emacs-pretest-bug <at> gnu.org
Subject: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Sat, 31 Jan 2009 21:13:03 +0200
Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the emacs-pretest-bug <at> gnu.org mailing list.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

In an Rmail buffer, type "C-x C-s" to save it: you will almost never
see the usual "Saving file FOO..." message, because buffer-soze,
called by save-buffer, returns the size of the current message, not of
the entire message collection.

If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
    `bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
d:/gnu/emacs/etc/DEBUG for instructions.


In GNU Emacs 23.0.60.1 (i386-mingw-nt5.1.2600)
 of 2009-01-31 on HOME-C4E4A596F7
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (3.4)'

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: ENU
  value of $XMODIFIERS: nil
  locale-coding-system: cp1255
  default-enable-multibyte-characters: t

Major mode: RMAIL

Minor modes in effect:
  tooltip-mode: t
  tool-bar-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  global-auto-composition-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
M-x l o a f d <backspace> <backspace> d - f i <tab> 
<return> l i s <tab> / f i l e s . e l <return> C-x 
C-f <up> <return> C-s s a v i n g SPC <help-echo> <help-echo> 
<help-echo> f i l e SPC <up> <up> <down> <down> C-s 
C-s C-r C-r C-r <up> M-x e d e b u <tab> d e <tab> 
<return> C-u M-x r m a i l <return> ~ / d a <tab> I 
N B O X . n e w <tab> <return> n p o d a <tab> f o 
o 1 2 3 4 5 . m b b <return> y e s <return> C-x C-s 
SPC SPC SPC SPC SPC SPC SPC G M-x r e p o r t <tab> 
<return>

Recent messages:
Added to d:/usr/eli/data/foo12345.mbb
 [2 times]
Result: t
 [3 times]
Result: 5470 (#o12536, #x155e, ?ᕞ)

Result: nil

Go-Nonstop...
Wrote d:/usr/eli/data/INBOX.new





Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs. (Sun, 01 Feb 2009 23:00:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to rms <at> gnu.org:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>. (Sun, 01 Feb 2009 23:00:04 GMT) Full text and rfc822 format available.

Message #10 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Richard M Stallman <rms <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>, 2137 <at> debbugs.gnu.org
Cc: emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60;
	Saving Rmail buffer does not show the "Saving file ..." message
Date: Sun, 01 Feb 2009 17:48:37 -0500
    In an Rmail buffer, type "C-x C-s" to save it: you will almost never
    see the usual "Saving file FOO..." message, because buffer-soze,
    called by save-buffer, returns the size of the current message, not of
    the entire message collection.

The clean way to fix this is with something like `buffer-swapped-with'.

I did not remember the existence of this feature when I recommended
implementing `buffer-swapped-with', so I can't really say "I told you
so."  But there is a general tendency that these sorts of features
need to be checked and acted on in various places in the code, and
most of them don't have any hooks.




bug reassigned from package `emacs' to `emacs,rmail'. Request was from Glenn Morris <rgm <at> gnu.org> to control <at> emacsbugs.donarmstrong.com. (Mon, 02 Feb 2009 07:20:04 GMT) Full text and rfc822 format available.

Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <emacs-devel <at> gnu.org>:
bug#2137; Package emacs,rmail. (Mon, 02 Feb 2009 17:15:07 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> iro.umontreal.ca>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <emacs-devel <at> gnu.org>. (Mon, 02 Feb 2009 17:15:07 GMT) Full text and rfc822 format available.

Message #17 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: rms <at> gnu.org
Cc: Eli Zaretskii <eliz <at> gnu.org>, 2137 <at> debbugs.gnu.org,
        emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Sun, 01 Feb 2009 20:35:12 -0500
>     In an Rmail buffer, type "C-x C-s" to save it: you will almost never
>     see the usual "Saving file FOO..." message, because buffer-soze,
>     called by save-buffer, returns the size of the current message, not of
>     the entire message collection.

> The clean way to fix this is with something like `buffer-swapped-with'.

> I did not remember the existence of this feature when I recommended
> implementing `buffer-swapped-with', so I can't really say "I told you
> so."  But there is a general tendency that these sorts of features
> need to be checked and acted on in various places in the code, and
> most of them don't have any hooks.

This problem at least simply points to a current bug in save-buffer.
Maybe with buffer-swapped-with you could work around the bug rather than
fixing it.  Or even fail to see that it's a general bug rather than
a problem with buffer-swapped-with.  I'm not sure it's a benefit.


        Stefan




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Mon, 02 Feb 2009 21:15:06 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Mon, 02 Feb 2009 21:15:06 GMT) Full text and rfc822 format available.

Message #22 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: rms <at> gnu.org, 2137 <at> debbugs.gnu.org, emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Mon, 02 Feb 2009 23:05:11 +0200
> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Cc: Eli Zaretskii <eliz <at> gnu.org>,  2137 <at> emacsbugs.donarmstrong.com,  emacs-devel <at> gnu.org
> Date: Sun, 01 Feb 2009 20:35:12 -0500
> 
> This problem at least simply points to a current bug in save-buffer.

Which bug is that?




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Tue, 03 Feb 2009 04:20:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Glenn Morris <rgm <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Tue, 03 Feb 2009 04:20:02 GMT) Full text and rfc822 format available.

Message #27 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Stefan Monnier <monnier <at> iro.umontreal.ca>, rms <at> gnu.org,
        2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Mon, 02 Feb 2009 23:12:23 -0500
Eli Zaretskii wrote:

>> This problem at least simply points to a current bug in save-buffer.
>
> Which bug is that?

Well, one thing I wonder about is: how does it know the correct buffer
_contents_, but not the correct buffer _size_...

(Ie, why isn't it overwriting your mailbox file with just the current
message you happen to be viewing at the time?)




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Tue, 03 Feb 2009 10:10:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to rms <at> gnu.org:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Tue, 03 Feb 2009 10:10:06 GMT) Full text and rfc822 format available.

Message #32 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Richard M Stallman <rms <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>, 2137 <at> debbugs.gnu.org
Cc: eliz <at> gnu.org, 2137 <at> debbugs.gnu.org, emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60;
	Saving Rmail buffer does not show the "Saving file ..." message
Date: Tue, 03 Feb 2009 04:59:36 -0500
    This problem at least simply points to a current bug in save-buffer.

It is a bug, for certain.  What I'm saying is that it is easy to fix
with something like `buffer-swapped-with', and hard to fix otherwise.




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Tue, 03 Feb 2009 19:40:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Tue, 03 Feb 2009 19:40:03 GMT) Full text and rfc822 format available.

Message #37 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: monnier <at> iro.umontreal.ca, rms <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Tue, 03 Feb 2009 21:31:59 +0200
> From: Glenn Morris <rgm <at> gnu.org>
> Cc: Stefan Monnier <monnier <at> iro.umontreal.ca>,  rms <at> gnu.org,  2137 <at> emacsbugs.donarmstrong.com
> Date: Mon, 02 Feb 2009 23:12:23 -0500
> 
> Eli Zaretskii wrote:
> 
> >> This problem at least simply points to a current bug in save-buffer.
> >
> > Which bug is that?
> 
> Well, one thing I wonder about is: how does it know the correct buffer
> _contents_, but not the correct buffer _size_...
> 
> (Ie, why isn't it overwriting your mailbox file with just the current
> message you happen to be viewing at the time?)

See rmail-write-region-annotate: it switches to the right buffer just
in time for write-region to save the correct text, but buffer-size is
called before that.

IMO, using annotations for the job of saving the mailbox is also a
horrible kludge.  All that just to avoid defining an Rmail-specific
binding for C-x C-s ...





Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Tue, 03 Feb 2009 19:50:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Tue, 03 Feb 2009 19:50:03 GMT) Full text and rfc822 format available.

Message #42 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: rms <at> gnu.org
Cc: monnier <at> iro.umontreal.ca, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60;	Saving Rmail buffer does not show the "Saving file ..." message
Date: Tue, 03 Feb 2009 21:39:44 +0200
> From: Richard M Stallman <rms <at> gnu.org>
> CC: eliz <at> gnu.org, 2137 <at> emacsbugs.donarmstrong.com, emacs-devel <at> gnu.org
> Date: Tue, 03 Feb 2009 04:59:36 -0500
> 
>     This problem at least simply points to a current bug in save-buffer.
> 
> It is a bug, for certain.  What I'm saying is that it is easy to fix
> with something like `buffer-swapped-with', and hard to fix otherwise.

Okay, I give up: what (or who) is `buffer-swapped-with', and how is it
related to this issue?

As for an alternative for fixing this, if the current kludgey way of
using annotations for switching to the right buffer behind
save-buffer's back is acceptable, we could add to
rmail-write-region-annotate another hack: a call to buffer-size
followed by the same message "Saving file ..." that should have been
displayed by save-buffer.




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Tue, 03 Feb 2009 21:35:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Tue, 03 Feb 2009 21:35:04 GMT) Full text and rfc822 format available.

Message #47 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Glenn Morris <rgm <at> gnu.org>, rms <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Tue, 03 Feb 2009 16:26:21 -0500
> IMO, using annotations for the job of saving the mailbox is also a
> horrible kludge.  All that just to avoid defining an Rmail-specific
> binding for C-x C-s ...

Thanks for your support,


        Stefan




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Tue, 03 Feb 2009 21:35:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Tue, 03 Feb 2009 21:35:05 GMT) Full text and rfc822 format available.

Message #52 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rms <at> gnu.org, 2137 <at> debbugs.gnu.org, emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Tue, 03 Feb 2009 16:29:08 -0500
>> This problem at least simply points to a current bug in save-buffer.
> Which bug is that?

The bug is that is uses buffer-size to determine the size of the file,
without taking into account the fact that
write-region-annotate-functions can add lot of stuff to the file that's
not in the buffer's text.

The simplest fix is to simply always output the message rather than only
for files larger than 50KB.
The more difficult one is to move the message from save-buffer to
write-region.


        Stefan




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 04:25:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 04:25:04 GMT) Full text and rfc822 format available.

Message #57 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: rms <at> gnu.org, 2137 <at> debbugs.gnu.org, emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 06:16:00 +0200
> From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
> Cc: rms <at> gnu.org, 2137 <at> emacsbugs.donarmstrong.com, emacs-devel <at> gnu.org
> Date: Tue, 03 Feb 2009 16:29:08 -0500
> 
> >> This problem at least simply points to a current bug in save-buffer.
> > Which bug is that?
> 
> The bug is that is uses buffer-size to determine the size of the file,
> without taking into account the fact that
> write-region-annotate-functions can add lot of stuff to the file that's
> not in the buffer's text.

How can buffer-size take that into account?  The correct full size of
what's written to the disk file is never seen, since write-region does
its job piecemeal, and the annotations are applied separately to each
piece by a_write.

Am I missing something?

> The simplest fix is to simply always output the message rather than only
> for files larger than 50KB.

That may be the only (non-kludgey) solution.

> The more difficult one is to move the message from save-buffer to
> write-region.

If I'm right above, this won't solve the case where annotations add a
lot.




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 04:30:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 04:30:03 GMT) Full text and rfc822 format available.

Message #62 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: rgm <at> gnu.org, rms <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 06:23:38 +0200
> From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
> Cc: Glenn Morris <rgm <at> gnu.org>, rms <at> gnu.org, 2137 <at> emacsbugs.donarmstrong.com
> Date: Tue, 03 Feb 2009 16:26:21 -0500
> 
> > IMO, using annotations for the job of saving the mailbox is also a
> > horrible kludge.  All that just to avoid defining an Rmail-specific
> > binding for C-x C-s ...
> 
> Thanks for your support,

Btw, isn't there's one more misfeature with write-region due to
rmail-write-region-annotate: if you use write-region to save a portion
of the current message to a file, would Rmail do that from the message
collection buffer instead? if so, what would be the encoding of
non-ASCII characters in the region as saved to the file?




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 18:35:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 18:35:03 GMT) Full text and rfc822 format available.

Message #67 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: 2137 <at> debbugs.gnu.org
Cc: monnier <at> IRO.UMontreal.CA, rms <at> gnu.org
Subject: Re: bug#2137: 23.0.60;	Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 20:29:17 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: rms <at> gnu.org, 2137 <at> emacsbugs.donarmstrong.com
> 
> Btw, isn't there's one more misfeature with write-region due to
> rmail-write-region-annotate: if you use write-region to save a portion
> of the current message to a file, would Rmail do that from the message
> collection buffer instead?

To answer myself: no, rmail-write-region-annotate does nothing if
write-region is invoked with its START argument non-nil (which means
to write only a portion of the current buffer).  So in this case, the
current message will be written to the file using
buffer-file-coding-system, as the user should expect.

Sorry for the noise.




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 18:35:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to rms <at> gnu.org:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 18:35:04 GMT) Full text and rfc822 format available.

Message #72 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Richard M Stallman <rms <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: eliz <at> gnu.org, rgm <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 13:27:49 -0500
    > IMO, using annotations for the job of saving the mailbox is also a
    > horrible kludge.  All that just to avoid defining an Rmail-specific
    > binding for C-x C-s ...

    Thanks for your support,

An Rmail-specific binding for C-x C-s is not enough to make saving
work correctly.  Users can save the file through C-x s also, and other
commands too.  The common mechanism for these commands is
`save-buffer'; whatever makes them save the right text has to work
inside there.

Eli wrote:

     we could add to
    rmail-write-region-annotate another hack: a call to buffer-size
    followed by the same message "Saving file ..." that should have been
    displayed by save-buffer.

The difficulty here is to display that message only in the case
where the hook was ultimately called from `save-buffer'.  This could
be done by making `save-buffer' bind some variable for such hooks
to check.

However, such fixes are less clean that implementing `buffer-swapped-with'
and checking it in the necessary places.  I previously implemented it in
`basic-save-buffer', but just moving the implementation to `save-buffer'
would make it DTRT for this.  The fix would add no complexity.

    The simplest fix is to simply always output the message rather than only
    for files larger than 50KB.

That would be easy, but not quite the most convenient behavior.

    The more difficult one is to move the message from save-buffer to
    write-region.





Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 19:55:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 19:55:04 GMT) Full text and rfc822 format available.

Message #77 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rms <at> gnu.org, 2137 <at> debbugs.gnu.org, emacs-devel <at> gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 14:46:49 -0500
>> The bug is that is uses buffer-size to determine the size of the file,
>> without taking into account the fact that
>> write-region-annotate-functions can add lot of stuff to the file that's
>> not in the buffer's text.
> How can buffer-size take that into account?

It can't.  Luckily we don't have to use buffer-size for that.

> The correct full size of what's written to the disk file is never
> seen, since write-region does its job piecemeal, and the annotations
> are applied separately to each piece by a_write.

Indeed.

> Am I missing something?

We do have all the annotations in a single place, so we can compute the
final total size before we start writing.

>> The simplest fix is to simply always output the message rather than only
>> for files larger than 50KB.
> That may be the only (non-kludgey) solution.

Yes, I find it to be the best solution.  I see no benefit in avoiding
the message for smaller files.


        Stefan




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 19:55:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> IRO.UMontreal.CA>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 19:55:05 GMT) Full text and rfc822 format available.

Message #82 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
To: rms <at> gnu.org
Cc: eliz <at> gnu.org, rgm <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 14:49:40 -0500
> An Rmail-specific binding for C-x C-s is not enough to make saving
> work correctly.  Users can save the file through C-x s also, and other
> commands too.  The common mechanism for these commands is
> `save-buffer'; whatever makes them save the right text has to work
> inside there.

save-buffer is not enough either in order to handle `autosave' (which
was the original reason for my using write-region-annotate-functions
rather than write-contents-functions).

>     The simplest fix is to simply always output the message rather than only
>     for files larger than 50KB.
> That would be easy, but not quite the most convenient behavior.

What would be the harm?


        Stefan




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 20:20:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 20:20:04 GMT) Full text and rfc822 format available.

Message #87 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: rms <at> gnu.org
Cc: monnier <at> IRO.UMontreal.CA, rgm <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 22:12:47 +0200
> From: Richard M Stallman <rms <at> gnu.org>
> CC: eliz <at> gnu.org, rgm <at> gnu.org, 2137 <at> emacsbugs.donarmstrong.com
> Date: Wed, 04 Feb 2009 13:27:49 -0500
> 
>     > IMO, using annotations for the job of saving the mailbox is also a
>     > horrible kludge.  All that just to avoid defining an Rmail-specific
>     > binding for C-x C-s ...
> 
>     Thanks for your support,
> 
> An Rmail-specific binding for C-x C-s is not enough to make saving
> work correctly.  Users can save the file through C-x s also, and other
> commands too.  The common mechanism for these commands is
> `save-buffer'; whatever makes them save the right text has to work
> inside there.

So we could introduce some mechanism specific for such situation, like
some hook that save-buffer would call or something.  Annotations just
aren't that mechanism, they exist to add text to the buffer before it
is written.

>      we could add to
>     rmail-write-region-annotate another hack: a call to buffer-size
>     followed by the same message "Saving file ..." that should have been
>     displayed by save-buffer.
> 
> The difficulty here is to display that message only in the case
> where the hook was ultimately called from `save-buffer'.

rmail-write-region-annotate uses its first argument to deduce that it
is being called to write the entire buffer.  Why couldn't the same
logic be used for the message?




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 20:20:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to Eli Zaretskii <eliz <at> gnu.org>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 20:20:05 GMT) Full text and rfc822 format available.

Message #92 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
Cc: rms <at> gnu.org, rgm <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 22:14:30 +0200
> From: Stefan Monnier <monnier <at> IRO.UMontreal.CA>
> Cc: eliz <at> gnu.org, rgm <at> gnu.org, 2137 <at> emacsbugs.donarmstrong.com
> Date: Wed, 04 Feb 2009 14:49:40 -0500
> 
> > An Rmail-specific binding for C-x C-s is not enough to make saving
> > work correctly.  Users can save the file through C-x s also, and other
> > commands too.  The common mechanism for these commands is
> > `save-buffer'; whatever makes them save the right text has to work
> > inside there.
> 
> save-buffer is not enough either in order to handle `autosave'

Why?  There's no need to autosave the buffer in which we show
messages, and the buffer where the whole collection is held, if it's
modified, will be autosaved by the usual machinery.  What am I
missing?




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Wed, 04 Feb 2009 23:30:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Stefan Monnier <monnier <at> iro.umontreal.ca>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Wed, 04 Feb 2009 23:30:02 GMT) Full text and rfc822 format available.

Message #97 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: rms <at> gnu.org, rgm <at> gnu.org, 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60; Saving Rmail buffer does not show the "Saving file ..." message
Date: Wed, 04 Feb 2009 18:21:39 -0500
> Why?  There's no need to autosave the buffer in which we show
> messages, and the buffer where the whole collection is held, if it's
> modified, will be autosaved by the usual machinery.  What am I
> missing?

The buffer where the `mbox' is held doesn't have a buffer-file-name
(the view buffer has buffer-file-name set instead, so without
write-region-annotate-functions, the autosave file contains the
currently shown message rather than a copy of the mbox).


        Stefan




Information forwarded to bug-submit-list <at> lists.donarmstrong.com, Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>:
bug#2137; Package emacs,rmail. (Thu, 05 Feb 2009 16:50:05 GMT) Full text and rfc822 format available.

Acknowledgement sent to rms <at> gnu.org:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>, Rmail Maintainers <bug-gnu-emacs <at> gnu.org>. (Thu, 05 Feb 2009 16:50:05 GMT) Full text and rfc822 format available.

Message #102 received at 2137 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Richard M Stallman <rms <at> gnu.org>
To: Stefan Monnier <monnier <at> IRO.UMontreal.CA>, 2137 <at> debbugs.gnu.org
Cc: 2137 <at> debbugs.gnu.org
Subject: Re: bug#2137: 23.0.60;
	Saving Rmail buffer does not show the "Saving file ..." message
Date: Thu, 05 Feb 2009 11:39:39 -0500
    save-buffer is not enough either in order to handle `autosave' (which
    was the original reason for my using write-region-annotate-functions
    rather than write-contents-functions).

That is true -- the implementation of saving for Rmail buffers needs
to handle auto-save one way or another.  That doesn't necessarily mean
it has to work through this hook.

    >     The simplest fix is to simply always output the message rather than only
    >     for files larger than 50KB.
    > That would be easy, but not quite the most convenient behavior.

    What would be the harm?

Nowadays, I guess it is ok to display the message for all files.




Reply sent to Eli Zaretskii <eliz <at> gnu.org>:
You have taken responsibility. (Sat, 07 Feb 2009 10:35:03 GMT) Full text and rfc822 format available.

Notification sent to Eli Zaretskii <eliz <at> gnu.org>:
bug acknowledged by developer. (Sat, 07 Feb 2009 10:35:04 GMT) Full text and rfc822 format available.

Message #107 received at 2137-done <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: rms <at> gnu.org, 2137-done <at> debbugs.gnu.org
Cc: monnier <at> IRO.UMontreal.CA
Subject: Re: bug#2137: 23.0.60;	Saving Rmail buffer does not show the "Saving file ..." message
Date: Sat, 07 Feb 2009 12:24:43 +0200
> Date: Thu, 05 Feb 2009 11:39:39 -0500
> Cc: 2137 <at> emacsbugs.donarmstrong.com
> 
>     >     The simplest fix is to simply always output the message rather than only
>     >     for files larger than 50KB.
>     > That would be easy, but not quite the most convenient behavior.
> 
>     What would be the harm?
> 
> Nowadays, I guess it is ok to display the message for all files.

That's what I did.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> emacsbugs.donarmstrong.com. (Sat, 07 Mar 2009 15:24:09 GMT) Full text and rfc822 format available.

This bug report was last modified 16 years and 192 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.