From: Eli Zaretskii <eliz@gnu.org>
To: npostavs@users.sourceforge.net
Cc: 27210-done@debbugs.gnu.org, ambrevar@gmail.com
Subject: bug#27210: 25.2; Recovering loaddefs.el with desktop-mode hangs when linum is on
Date: Sun, 04 Jun 2017 19:31:34 +0300 [thread overview]
Message-ID: <83zidnaf4p.fsf@gnu.org> (raw)
In-Reply-To: <87efuzzt6c.fsf@users.sourceforge.net> (npostavs@users.sourceforge.net)
> From: npostavs@users.sourceforge.net
> Cc: 27210@debbugs.gnu.org, ambrevar@gmail.com
> Date: Sun, 04 Jun 2017 11:08:59 -0400
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
> > And don't forget that the initial frame is invisible in non-daemon
> > sessions as well, until some point during startup.
>
> I don't understand, is this an objection?
Sorry for being unclear: I meant to point out that a non-daemon
startup initially has such a frame as well, and we never heard any
complaints about that. Which might mean that some of the code
routinely run during startup expects to find that frame marked
visible.
> > My alternative proposal is much simpler, is localized to linum.el, and
> > in a nutshell tests exactly the same condition, since any frame in a
> > daemon session that can be visible is by definition a client frame.
> > Do you see any disadvantages with installing that instead?
>
> The only disadvantage is that we still have this invisible daemon frame
> which is marked as visible. I agree it's okay to apply your patch now
> and see if we get some other similar problems later.
OK, I've pushed the change. Let's keep an eye on similar problems if
they pop up.
prev parent reply other threads:[~2017-06-04 16:31 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-03 14:29 bug#27210: 25.2; Recovering loaddefs.el with desktop-mode hangs when linum is on Pierre Neidhardt
2017-06-03 14:56 ` Eli Zaretskii
2017-06-03 16:22 ` Pierre Neidhardt
2017-06-03 17:43 ` npostavs
2017-06-03 18:02 ` Eli Zaretskii
2017-06-03 18:38 ` Eli Zaretskii
2017-06-03 19:52 ` npostavs
2017-06-03 23:07 ` npostavs
2017-06-04 14:15 ` Eli Zaretskii
2017-06-04 15:08 ` npostavs
2017-06-04 16:31 ` Eli Zaretskii [this message]
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=83zidnaf4p.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=27210-done@debbugs.gnu.org \
--cc=ambrevar@gmail.com \
--cc=npostavs@users.sourceforge.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 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.