bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#6415: [PATCH] fix edebug instrumentation of dotted pairs in macros


From: Gemini Lasswell
Subject: bug#6415: [PATCH] fix edebug instrumentation of dotted pairs in macros
Date: Sun, 05 Nov 2017 13:45:21 -0800
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.0.90 (gnu/linux)

Here's a new patch for this bug, based on the ideas in Steve's patch.
It also allows &rest and specs wrapped in vectors to attempt to match
before a dotted tail.

Attachment: 0001-Fix-Edebug-s-handling-of-dotted-specs-bug-6415.patch
Description: Text document

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> This edebug-dotted-spec business is really ugly, I wonder if/how we
> could just get rid of this variable.  Or at least document clearly what
> it is supposed to mean.

I agree. After far too many hours of looking at this, I think the way
to get rid of the variable is to make Edebug's internal representation
of its specs into a cl-defstruct so there is room for a "dotted" flag,
and then change all the edebug match code so that whenever it makes a
new spec or modifies the one it is working with, it inherits the
dotted flag. I have a branch with this partially done and am confident
that it will work and not cause a performance problem, but it seems
like a lot of work and a lot of changes to stable code to make it work
exactly the same. So I added a docstring.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]