Re: Edebug: avoid messages on 'f' command

[Paul Pogonyshev]

By the way, here are some other things that I miss with Edebug. Maybe
you could find time to do something about these.

* Highlight breakpoints with overlays and/or on fringe.

* When setting a breakpoint in a non-instrumented form, instrument it
for me rather than printing an error.

* When C-M-x a definition that has breakpoints, try to preserve them,
rather than resetting. It's especially annoying to lose conditional
breakpoints. Might be hard to implement, though, as it's not clear how
to compare old and new definitions to decide where to move
breakpoints.

* Some way to list active breakpoints.

* Add analogue of 'f' (maybe bind to 'F') that temporary disables all
breakpoints until its destination is reached. Maybe also add a command
to "go to point ignoring all breakpoints".

Paul


On 5 May 2016 at 23:44, Paul Pogonyshev <address@hidden> wrote:
> Yes, it works fine for me. Thank you.
>
> Paul
>
> On 5 May 2016 at 15:10, Alan Mackenzie <address@hidden> wrote:
>> Hello, Paul.
>>
>> On Tue, May 03, 2016 at 08:58:52PM +0200, Paul Pogonyshev wrote:
>>> In Edebug 'f' is bound to 'edebug-forward-sexp'. It can be seen as
>>> "faster" space key, allowing you to step through function a whole sexp
>>> at a time. However, there is an annoyance to it: each time I hit it,
>>> echo area displays "Break" for about a second, which is only then
>>> replaced by "Result: ...". By comparison, space displays result
>>> immediately.
>>
>> Yes, this irritates me too.
>>
>>> Request: improve Edebug to not print such intermediate message(s)
>>> during 'f' and similar commands.
>>
>> Would you please try out the following patch.  It entirely cuts out the
>> one second pause on all "break commands" (whatever they might be).
>> Hopefully there aren't any unwanted side effects.
>>
>> Just set the new customisable variable to nil, and off you go!
>>
>> The patch is based on today's master branch.
>>
>>
>>
>> diff --git a/lisp/emacs-lisp/edebug.el b/lisp/emacs-lisp/edebug.el
>> index e8484fa..5d2e4e6 100644
>> --- a/lisp/emacs-lisp/edebug.el
>> +++ b/lisp/emacs-lisp/edebug.el
>> @@ -233,6 +233,11 @@ edebug-sit-for-seconds
>>    :type 'number
>>    :group 'edebug)
>>
>> +(defcustom edebug-sit-on-break t
>> +  "Whether or not to pause for `edebug-sit-for-seconds' on reaching a break"
>> +  :type 'boolean
>> +  :group 'edebug)
>> +
>>  ;;; Form spec utilities.
>>
>>  (defun get-edebug-spec (symbol)
>> @@ -2489,6 +2494,7 @@ edebug--display-1
>>                  (progn
>>                    ;; Display result of previous evaluation.
>>                    (if (and edebug-break
>> +                           edebug-sit-on-break
>>                             (not (eq edebug-execution-mode 'Continue-fast)))
>>                        (sit-for edebug-sit-for-seconds)) ; Show message.
>>                    (edebug-previous-result)))
>>
>>
>>> Paul
>>
>> --
>> Alan Mackenzie (Nuremberg, Germany).