unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: martin rudalics <rudalics@gmx.at>
Cc: lekktu@gmail.com, 14062@debbugs.gnu.org
Subject: bug#14062: 24.3.50; emacs_backtrace.txt
Date: Mon, 15 Apr 2013 17:18:18 +0300	[thread overview]
Message-ID: <83y5cj6fo5.fsf@gnu.org> (raw)
In-Reply-To: <516BF54E.9020802@gmx.at>

> Date: Mon, 15 Apr 2013 14:40:46 +0200
> From: martin rudalics <rudalics@gmx.at>
> CC: drew.adams@oracle.com, Juanma Barranquero <lekktu@gmail.com>, 
>  14062@debbugs.gnu.org
> 
>  > If XBUFFER is indeed the problem, then this means that this snippet,
>  > around line 3115 of w32fns.c:
>  >
>  > 	  f = x_window_to_frame (dpyinfo, hwnd);
>  > 	  w = XWINDOW (FRAME_SELECTED_WINDOW (f));
>  >
>  > produces a non-leaf window in w.  Can a frame's selected window be
>  > non-leaf?
> 
> I could imagine lots of things including dead windows.

Are these "things", including dead windows, allowed to be the selected
window of a frame that gets input messages from Windows, i.e. is at
least visible, if not in the foreground?

> But it would be a strange coincidence if it were a non-leaf window.
> What drives you to this question?

Only a non-leaf window can have its w->contents be something other
than a buffer, right?  If BUFFERP(w->contents) returns zero and
XBUFFER hits an assertion violation, what else can this window be
except non-leaf?

>  > Anyway, I added in trunk revision 112287 some more debugging code to
>  > point out which assertions are violated.  Let's see what that gets us.
> 
> But you also added some parentheses so now we might not be able to find
> out whether it was just due to badly written macros ;-)

I don't think so.  I examined the preprocessed source, and didn't see
any instance of missing parentheses.  I added some just so someone who
looks at the macros won't wonder, like I did, whether this could be
the problem.

But even if you are right, and the problem will now disappear, we can
still resolve this bug by simply going back to the original code.





  reply	other threads:[~2013-04-15 14:18 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-26 23:33 bug#14062: 24.3.50; emacs_backtrace.txt Drew Adams
2013-03-27  6:57 ` Eli Zaretskii
2013-03-27  9:45   ` Dani Moncayo
2013-03-27 12:20     ` Eli Zaretskii
2013-03-27 13:39       ` Drew Adams
2013-03-28  9:25         ` Eli Zaretskii
2013-04-15  7:35           ` Eli Zaretskii
2013-04-15 11:54             ` Juanma Barranquero
2013-04-15 12:30               ` Eli Zaretskii
2013-04-15 12:40             ` martin rudalics
2013-04-15 14:18               ` Eli Zaretskii [this message]
2013-04-15 15:53                 ` martin rudalics
2013-04-15 16:21                   ` Eli Zaretskii
2013-04-15 19:22                     ` martin rudalics
2013-04-16  6:08                       ` Eli Zaretskii
2013-04-22 16:04                         ` Drew Adams
2013-04-22 16:12                           ` Juanma Barranquero
2013-04-22 18:05                             ` Eli Zaretskii
2013-04-22 18:18                               ` Drew Adams
2013-05-04 10:27                               ` Eli Zaretskii
2013-05-04 12:27                                 ` martin rudalics
2013-05-04 12:33                                   ` Eli Zaretskii
2013-05-04 12:45                                     ` martin rudalics
2013-05-04 13:18                                       ` Eli Zaretskii
2013-05-04 18:59                                         ` martin rudalics
2013-05-04 14:38                                 ` Drew Adams
2013-03-27 13:37   ` Drew Adams

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=83y5cj6fo5.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=14062@debbugs.gnu.org \
    --cc=lekktu@gmail.com \
    --cc=rudalics@gmx.at \
    /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).