unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: mark.lillibridge@hp.com
Cc: 9831@debbugs.gnu.org, jpff <jpff@codemist.co.uk>
Subject: bug#9831: cause of bug found!  [PATCH]
Date: Mon, 24 Oct 2011 11:31:59 +0200	[thread overview]
Message-ID: <4EA5308F.2050608@gmx.at> (raw)
In-Reply-To: <qmh7h3vh31a.fsf@hp.com>

 >     Sorry, more background.  The bug OP and I am reporting is as
 > follows: we have two Rmail buffers, say A and B, each with summary
 > buffers.  However, only A and it's summary are displayed in windows.  We
 > then output the current message from A to B via 'o'.  The bug is that at
 > this point the summary for B becomes displayed when it should not.

I'm probably too silly to understand.  John was talking about "o" not
doing the right thing, but IIUC "o" calls `rmail-output' and not
`rmail-summary-output' in his case.  At least that's what I deduct from
his "When reading mail o writes the message to another file, or buffer
if it is loaded" and the doc-string of `rmail-output' saying "Append
this message to mail file FILE-NAME".  Then John says that "It also
changes to that buffer and this seriously interferes with work flow, as
it is inconsistent with when the file is not in a buffer" but
unfortunately I don't understand what "changes to that buffer" means in
this context.

Moreover, John was saying that "This seems fairly recent behaviour and
is causing significant problems" but I don't find any recent reference
to a change of `rmail-summary' in the Logs.  Finally, John nowhere
talked about point moving to some inconvenient position.  John could you
please clarify these issues?

 >     Why?  The filing code updates the summary for the buffer the
 > messages being filed to (e.g., B) so that it shows the message just
 > added to that buffer if appropriate.  This should not cause that summary
 > to be displayed but it does due to the bug.
 >
 >     Why?  The summary is updated via (rmail-update-summary).
 > Historically, this does not cause the updated buffer to be displayed,

Can you tell me when and where this was changed?

 > but because of the bug if this summary was produced by rmail-summary, it
 > will be displayed.
 >
 >     Why? rmail-update-summary makes a saved function call (depending on
 > the filtering requested, a different call is necessary to rebuild the
 > summary) to update the summary.  If the summary was originally created via
 > rmail-summary, then the saved call is (rmail-summary), which because of
 > the bug displays the summary.
 >
 >     Why?  Because someone incorrectly added code to display the summary
 > buffer on summary update to rmail-summary.

According to our Logs `rmail-update-summary' hasn't been changed for
many years.

 >     I changed the code so that rmail-summary when called by the user
 > (e.g., via 'h') does always display the summary but does not do so when
 > called via rmail-update-summary.
 >
 >     Is this more clear?  I think the part you were unclear about is that
 > there are two Rmail buffers involved, each with their own summary.

I still suppose your's is a different bug.  But I suspect that any of
these bugs may have its cause in a recent change of the buffer display
routines.  Unfortunately, I'm not of much help here since I don't use
rmail.

martin





  reply	other threads:[~2011-10-24  9:31 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-22 11:08 bug#9831: 24.0.90; o and c-o in RMAIL change buffer john ffitch
2011-10-22 20:06 ` bug#9831: narrowing the bug down Mark Lillibridge
2011-10-22 20:45   ` Mark Lillibridge
2011-10-22 21:26     ` bug#9831: cause of bug found! [PATCH] Mark Lillibridge
2011-10-23  9:19       ` martin rudalics
2011-10-23 20:21         ` Mark Lillibridge
2011-10-24  9:31           ` martin rudalics [this message]
2011-10-27  2:53             ` Mark Lillibridge
2011-10-27  9:52               ` martin rudalics
2011-10-27  3:09 ` bug#9831: Your bug report re: o and c-o in RMAIL change buffer Mark Lillibridge
2011-11-14  9:32   ` bug#9831: " Glenn Morris

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EA5308F.2050608@gmx.at \
    --to=rudalics@gmx.at \
    --cc=9831@debbugs.gnu.org \
    --cc=jpff@codemist.co.uk \
    --cc=mark.lillibridge@hp.com \
    /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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).