all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Alan Mackenzie <acm@muc.de>
Cc: emacs-devel@gnu.org
Subject: Re: Making Emacs Lisp easier to debug
Date: Sat, 11 Nov 2023 18:01:08 +0200	[thread overview]
Message-ID: <831qcwwa7v.fsf@gnu.org> (raw)
In-Reply-To: <ZU-WERAKIaF5D5-V@ACM> (message from Alan Mackenzie on Sat, 11 Nov 2023 14:56:17 +0000)

> Date: Sat, 11 Nov 2023 14:56:17 +0000
> Cc: emacs-devel@gnu.org
> From: Alan Mackenzie <acm@muc.de>
> 
> > > With font lock, or any other Lisp hook called from redisplay, it should
> > > be possible, in a recursive-edit loop, to run edebug, displaying on a
> > > different frame.  That different frame would be running in the inner
> > > redisplay while the outer redisplay would be suspended.
> 
> > Why do you need an inner redisplay for that?
> 
> Because the outer redisplay would be the thing being debugged, and hence
> not in a position to display the progress of edebug.

But you don't want to debug redisplay, you want to debug the font-lock
code called by redisplay.

> > And what will that frame show, given that the outer redisplay is
> > halfway through fontifying the text? what do you expect to see there,
> > and why?
> 
> I think the scope of a redisplay operation could change from everything
> to a single frame.

Not really, not with the current code design.

And you don't want a separate frame, you want a separate window.

> While stepping through a hook in edebug in the inner redisplay, the
> outer redisplay would (I think) carry on looking like it did before the
> outer redisplay started.  Or, possibly, it might look like a bare frame,
> I'm not sure.

So basically, the outer redisplay doesn't exist.  All you need is for
it to call the font-lock code.  Which once again brings me to the
question: why isn't jit-lock-debug-mode not what you want?

> > > > Re-entering redisplay in the middle of a redisplay cycle means that
> > > > the outer redisplay didn't finish preparing the glyph matrices, and
> > > > what do you want the inner redisplay to do in such a case?
> 
> > > Work with the glyph matrices belonging to the inner redisplay whilst the
> > > outer one is suspended.
> 
> > But that will immediately get you into the same problem, since the
> > offending window will get redisplayed by the inner redisplay, and will
> > again cause Edebug, etc., ad nauseam.
> 
> We could put a limit on the nesting depth of redisplay nesting that
> edebug would cope with, a small integer (2 or 3, probably).

That will prevent infinite recursion, but it won't solve your problem,
because you do want the "inner" redisplay to finish, so it shows the
window where you are running the debugger.

> > .... and what you call "inner redisplay" is not what you want at all.
> 
> I don't understand why you say this.

I'm saying that what you seem to want is an ability to display a
single window, the one where you run the debugger.  All the rest is of
no interest to you, and can, for example, be stopped in its tracks.
The solution to that is not a reentrant redisplay, it's something
else.



  reply	other threads:[~2023-11-11 16:01 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-10 20:56 Making Emacs Lisp easier to debug Alan Mackenzie
2023-11-11  6:52 ` Eli Zaretskii
2023-11-11  9:01   ` Ihor Radchenko
2023-11-11 11:04   ` Alan Mackenzie
2023-11-11 11:10     ` Eli Zaretskii
2023-11-11 12:10       ` Alan Mackenzie
2023-11-11 13:47         ` Eli Zaretskii
2023-11-11 14:56           ` Alan Mackenzie
2023-11-11 16:01             ` Eli Zaretskii [this message]
2023-11-11 17:23               ` Alan Mackenzie
2023-11-11 17:54                 ` Eli Zaretskii
2023-11-11 19:55                   ` Alan Mackenzie
2023-11-12  7:17                     ` Eli Zaretskii
2023-11-12 12:08                       ` Alan Mackenzie
2023-11-12 12:28                         ` Eli Zaretskii
2023-11-13 18:20                       ` XY Problems (tangent, related to common discussion issue here) chad

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=831qcwwa7v.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=acm@muc.de \
    --cc=emacs-devel@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 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.