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

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

[Ludovic Brenta]

X-Debbugs-CC: ada-mode-users@nongnu.org
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.

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

[WAROQUIERS Philippe via Bug reports for GNU Emacs, the Swiss army knife of text editors]

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=eurocontrol.int@nongnu.org> On Behalf Of
> Ludovic Brenta
> Sent: 19 November 2019 15:58
> To: 38270@debbugs.gnu.org
> Cc: ada-mode-users@nongnu.org
> Subject: bug#38270: 26.1; ada-mode 6.2.1 Ada > Misc > Reset Parser appears
> useless
>
> X-Debbugs-CC: ada-mode-users@nongnu.org
> 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.

bug#38270: update

[Stephen Leake]

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

bug#38270: update

[Stephen Leake]

fixed in 7.0.1
-- 
-- Stephe

bug#38270: closed by ada-mode version 7.0.1

[Stephen Leake]

-- 
-- Stephe