From: Ken Brown <kbrown@cornell.edu>
To: Eli Zaretskii <eliz@gnu.org>, Richard Copley <rcopley@gmail.com>
Cc: 25875-done@debbugs.gnu.org
Subject: bug#25875: 26.0.50; Hang logging out of MS-Windows
Date: Tue, 28 Feb 2017 11:40:05 -0500 [thread overview]
Message-ID: <7708fd40-e6dd-8022-4acf-9c86630cce19@cornell.edu> (raw)
In-Reply-To: <834lzepbtp.fsf@gnu.org>
On 2/28/2017 10:36 AM, Eli Zaretskii wrote:
>> From: Richard Copley <rcopley@gmail.com>
>> In the nine months since Ken's change, I've noticed a problem with
>> Emacs twice.
>
> Then I guess it would be better to just push those changes and see if
> they do any harm.
>> I can't get my head around the idea. If we don't care whether or not
>> the action in question has finished when SendMessage returns,
>> then why are we using SendMessage? And if we do care, then
>> shouldn't I expect weird bugs caused by timing out when the system's
>> under load?
>
> If a window procedure doesn't process messages for more than 5 sec,
> Windows will put "Not Responding" on its caption bar. So I think the
> 100 msec number in Ken's patch should be changed to something like
> 6000, and then we are fine, because even on a busy system this should
> be long enough. And if Emacs (and the OS) is about to shut down, it's
> even less of a problem to ignore a message that timed out, IMO.
I've pushed the patch (with 6000 msec) and am closing the bug. Richard,
please reopen if you ever see the problem again.
I'm leaving the sleep(1000) in w32_wnd_proc, at least for now, because I
want to find out about it if the problem isn't really fixed. Maybe I'll
add a FIXME comment.
Ken
next prev parent reply other threads:[~2017-02-28 16:40 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-25 19:35 bug#25875: 26.0.50; Hang logging out of MS-Windows Richard Copley
2017-02-25 19:41 ` Richard Copley
2017-02-25 20:36 ` Eli Zaretskii
2017-02-25 21:13 ` Richard Copley
2017-02-26 15:47 ` Eli Zaretskii
2017-02-26 18:26 ` Richard Copley
2017-02-25 20:33 ` Eli Zaretskii
2017-02-25 21:07 ` Richard Copley
2017-02-25 21:30 ` Richard Copley
2017-02-25 21:37 ` Richard Copley
2017-02-25 22:02 ` Richard Copley
2017-02-26 15:47 ` Eli Zaretskii
2017-02-26 18:37 ` Richard Copley
2017-02-26 15:44 ` Eli Zaretskii
2017-02-26 18:04 ` Ken Brown
2017-02-26 18:25 ` Eli Zaretskii
2017-02-26 18:58 ` Ken Brown
2017-02-26 19:25 ` Eli Zaretskii
2017-02-26 23:38 ` Ken Brown
2017-02-27 8:14 ` Richard Copley
2017-02-27 15:36 ` Eli Zaretskii
2017-02-27 19:04 ` Richard Copley
2017-02-27 19:16 ` Eli Zaretskii
2017-02-27 19:23 ` Richard Copley
2017-02-27 19:30 ` Richard Copley
2017-02-27 19:39 ` Ken Brown
2017-02-27 19:46 ` Richard Copley
2017-02-27 19:56 ` Richard Copley
2017-02-27 20:19 ` Ken Brown
2017-02-27 20:27 ` Eli Zaretskii
2017-02-27 20:52 ` Richard Copley
2017-02-27 20:58 ` Eli Zaretskii
2017-02-27 21:09 ` Richard Copley
2017-02-28 3:30 ` Eli Zaretskii
2017-02-28 6:37 ` Richard Copley
2017-02-27 22:37 ` Ken Brown
2017-02-27 23:03 ` Richard Copley
2017-02-28 3:35 ` Eli Zaretskii
2017-02-28 7:21 ` Richard Copley
2017-02-28 15:36 ` Eli Zaretskii
2017-02-28 16:40 ` Ken Brown [this message]
2017-02-28 16:44 ` Eli Zaretskii
2017-02-28 18:59 ` Richard Copley
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=7708fd40-e6dd-8022-4acf-9c86630cce19@cornell.edu \
--to=kbrown@cornell.edu \
--cc=25875-done@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=rcopley@gmail.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 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.