From: Drew Adams <drew.adams@oracle.com>
To: Paul Eggert <eggert@cs.ucla.edu>, Glenn Morris <rgm@gnu.org>
Cc: Richard Munitz <rmunitz1@bloomberg.net>, 20247@debbugs.gnu.org
Subject: bug#20247: 24.4; Emacs hangs at startup in desktop mode
Date: Fri, 20 May 2016 08:54:59 -0700 (PDT) [thread overview]
Message-ID: <4a60533f-2b56-4187-8bd9-78cb4e032684@default> (raw)
In-Reply-To: <866b4d59-3fe0-4086-cec5-0292be35a2a5@cs.ucla.edu>
> > A better fix, I think, would be to do something like this:
>
> If someone has the time to develop an even-better patch, that would be
> great. In the meantime, the question is whether the proposed patch
> improves Emacs.
All your patch does is make the _default_ value be one that doesn't
manifest the bug. That does not sound like a fix, to me. I would
hardly even characterize it as an improvement, though admittedly
some users might be less likely to stumble on the bug.
To me, this is, at best, papering over the cracks - and the paper
is very thin.
I certainly hope that the bug remains open, so this can be fixed,
whether or not your patch gets applied. The bug will still exist.
> I don't use desktop mode and don't know the pros and
> cons of the various settings. That being said, it appears that
> defaulting desktop-restore-in-current-display to nil caused significant
> misbehavior for a real use case, whereas defaulting to t would have
> worked for that use case. There is something to be said for choosing a
> default that is less likely to cause a bug, even if you have not fixed
> the bug in general.
There might be something to be said for that, but it is not much, IMO.
My recommendation would be to leave the default value as it is and
leave the bug open until someone can debug it and fix it. This bug
should not be a reason behind the default-value choice.
What's the hurry to close bugs that would motivate not fixing them?
next prev parent reply other threads:[~2016-05-20 15:54 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-02 15:22 bug#20247: 24.4; Emacs hangs at startup in desktop mode Richard Munitz (BLOOMBERG/ 731 LEX)
2015-04-15 18:03 ` Glenn Morris
2015-04-16 13:46 ` Richard Munitz (BLOOMBERG/ 731 LEX)
2016-05-20 15:08 ` Paul Eggert
2016-05-20 15:24 ` Drew Adams
2016-05-20 15:34 ` Paul Eggert
2016-05-20 15:54 ` Drew Adams [this message]
2016-05-20 16:10 ` Eli Zaretskii
2016-05-20 16:31 ` Paul Eggert
2016-05-20 16:09 ` Eli Zaretskii
2016-05-20 16:06 ` Eli Zaretskii
[not found] ` <<83d1og90iw.fsf@gnu.org>
2016-05-20 17:15 ` Drew Adams
2016-05-20 17:44 ` Eli Zaretskii
[not found] ` <<<83d1og90iw.fsf@gnu.org>
[not found] ` <<4eebea73-4fcb-4dc7-b149-cef7a34a3c16@default>
[not found] ` <<837feo8w0u.fsf@gnu.org>
2016-05-20 17:54 ` Drew Adams
2016-05-23 16:53 ` Paul Eggert
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=4a60533f-2b56-4187-8bd9-78cb4e032684@default \
--to=drew.adams@oracle.com \
--cc=20247@debbugs.gnu.org \
--cc=eggert@cs.ucla.edu \
--cc=rgm@gnu.org \
--cc=rmunitz1@bloomberg.net \
/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).