bug#38270: 26.1; ada-mode 6.2.1 Ada > Misc > Reset Parser appears useless

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

bug#38270: 26.1; ada-mode 6.2.1 Ada > Misc > Reset Parser appears useless

Ludovic Brenta
X-Debbugs-CC: [hidden email]
Package: emacs, ada-mode
Version: 26.1
Severity: minor

This is a followup to #37620.  One of my colleagues reports being
hit by it quite often (this is indeed the most annoying bug remaining
in ada-mode right now).  When this happens, he tries Ada > Misc >
Reset Parser but this does not help the situation.  Only M-x
wisi-kill-parser makes emacs responsive again.  Therefore, I suggest
replacing the menu entry with Ada > Misc > Kill Parser.

I'm not sure what "Reset Parser" is supposed to do anyway.

--
Ludovic Brenta.



Reply | Threaded
Open this post in threaded view
|

bug#38270: 26.1; ada-mode 6.2.1 Ada > Misc > Reset Parser appears useless

Emacs - Bugs mailing list
Maybe 'Restart Parser' ?
(kill parser looks too nasty).

For sure, we better have a 'please repair the parser' which works,
rather than having a (maybe microscopically) more efficient 'Reset parser'
that (at least in my case) does not work most of the time.

Philippe


> -----Original Message-----
> From: Ada-mode-users <ada-mode-users-
> bounces+philippe.waroquiers=[hidden email]> On Behalf Of
> Ludovic Brenta
> Sent: 19 November 2019 15:58
> To: [hidden email]
> Cc: [hidden email]
> Subject: bug#38270: 26.1; ada-mode 6.2.1 Ada > Misc > Reset Parser appears
> useless
>
> X-Debbugs-CC: [hidden email]
> Package: emacs, ada-mode
> Version: 26.1
> Severity: minor
>
> This is a followup to #37620.  One of my colleagues reports being
> hit by it quite often (this is indeed the most annoying bug remaining
> in ada-mode right now).  When this happens, he tries Ada > Misc >
> Reset Parser but this does not help the situation.  Only M-x
> wisi-kill-parser makes emacs responsive again.  Therefore, I suggest
> replacing the menu entry with Ada > Misc > Kill Parser.
>
> I'm not sure what "Reset Parser" is supposed to do anyway.
>
> --
> Ludovic Brenta.
>
>

____

This message and any files transmitted with it are legally privileged and intended for the sole use of the individual(s) or entity to whom they are addressed. If you are not the intended recipient, please notify the sender by reply and delete the message and any attachments from your system. Any unauthorised use or disclosure of the content of this message is strictly prohibited and may be unlawful.

Nothing in this e-mail message amounts to a contractual or legal commitment on the part of EUROCONTROL, unless it is confirmed by appropriately signed hard copy.

Any views expressed in this message are those of the sender.



Reply | Threaded
Open this post in threaded view
|

bug#38270: update

Stephen Leake-3
In reply to this post by Ludovic Brenta
The menu entry "Reset Parser" runs wisi-reset-parser, which clears all
of the various caches associated with the parser. It is useful when an
elisp bug is causing problems.

wisi-kill-parser kills the parser process, which is useful when the
parser is taking too long due to a complex error, or for Ada bugs, and
then cleans up just two of the parse caches.

I'll change wisi-kill-parser to call wisi-reset-parser, so it handles
both elisp and Ada bugs, and change the menu entry to "Restart Parser"
and have it call wisi-kill-parser.

--
-- Stephe