unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: bojohan+news@dd.chalmers.se (Johan Bockgård)
To: bug-gnu-emacs@gnu.org
Subject: Re: appointment display during isearch replaces buffer contents with history-element
Date: Tue, 26 Feb 2008 12:53:02 +0100	[thread overview]
Message-ID: <yoijlk582au9.fsf@remote2.student.chalmers.se> (raw)
In-Reply-To: yoijfxx59al1.fsf@remote1.student.chalmers.se

bojohan+news@dd.chalmers.se (Johan Bockgård) writes:

> Chong Yidong <cyd@stupidchicken.com> writes:
>
>>          an alike.  Make read_key_sequence aware of that.  */
>>            if (timers_run != old_timers_run
> [...]
>
> There are two occurrences of this in wait_reading_process_output. Should
> the other be changed too?

Anyone?  I'm talking about this change:

2008-01-10  Chong Yidong  <cyd@stupidchicken.com>

        * process.c (wait_reading_process_output): Check for window
        changes caused by timers.

wait_reading_process_output contains two conditionals that may run
record_asynch_buffer_change.  The check for window changes was only
added to the latter one:

process.c:4368

    /* If a timer has run, this might have changed buffers
       an alike.  Make read_key_sequence aware of that.  */
    if (timers_run != old_timers_run
        && old_buffer != current_buffer
        && waiting_for_user_input_p == -1)
      record_asynch_buffer_change ();

process.c:4692

    /* If a timer has run, this might have changed buffers
       an alike.  Make read_key_sequence aware of that.  */
    if (timers_run != old_timers_run
        && waiting_for_user_input_p == -1
        && (old_buffer != current_buffer
        || !EQ (old_window, selected_window)))
      record_asynch_buffer_change ();


Is this intentional?

-- 
Johan Bockgård





  reply	other threads:[~2008-02-26 11:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-14 14:35 appointment display during isearch replaces buffer contents with history-element Martin Fischer
2007-11-17 10:44 ` Johan Bockgård
2007-11-17 11:04   ` martin rudalics
2007-11-18 22:43     ` Johan Bockgård
     [not found] ` <mailman.3706.1195296268.18990.bug-gnu-emacs@gnu.org>
2007-11-17 17:10   ` Martin Fischer
2007-11-19  0:37 ` Johan Bockgård
2008-01-02 14:36   ` Chong Yidong
2008-01-10  4:10     ` Glenn Morris
2008-01-10 11:36     ` Johan Bockgård
2008-02-26 11:53       ` Johan Bockgård [this message]
2008-02-26 21:31         ` Chong Yidong

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=yoijlk582au9.fsf@remote2.student.chalmers.se \
    --to=bojohan+news@dd.chalmers.se \
    --cc=bug-gnu-emacs@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 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).