GNU bug report logs - #62508
28.2; vhdl-mode ModelSim compile warnings aren't highlighted

Previous Next

Package: emacs;

Reported by: Cyril Arnould <cyril.arnould <at> outlook.com>

Date: Tue, 28 Mar 2023 19:49:01 UTC

Severity: normal

Found in version 28.2

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #28 received at 62508 <at> debbugs.gnu.org (full text, mbox):

From: Reto Zimmermann <reto <at> gnu.org>
To: Cyril Arnould <cyril.arnould <at> outlook.com>,
 Mattias Engdegård <mattias.engdegard <at> gmail.com>
Cc: "friedrich.beckmann <at> gmx.de" <friedrich.beckmann <at> gmx.de>,
 Eli Zaretskii <eliz <at> gnu.org>, "62508 <at> debbugs.gnu.org" <62508 <at> debbugs.gnu.org>
Subject: Re: bug#62508: 28.2; vhdl-mode ModelSim compile warnings aren't
 highlighted
Date: Mon, 3 Apr 2023 12:16:38 +0200
[Message part 1 (text/plain, inline)]
The regexp I sent had the "+" in there, it must have gotten lost 
somewhere in the process.  The single-escaped square brackets always 
worked fine, but I agree that it should be double-escaped to conform to 
the standard.

So yes, I agree with Mattias' fix.

Reto


On 03.04.2023 09:21, Cyril Arnould wrote:
> True, looking at the new regexp in emacs the escape characters are 
> highlighted in red, indicating an error. I can verify Mattias' 
> modification still works for me.
>
> I can't properly verify the bracketed version of the warning since I 
> don't get those with my ModelSim, but if I manually type out a warning 
> with brackets it does get highlighted when using re-builder. With 
> Reto's modification on the other hand the bracket warning is not 
> highlighted.
>
> Gesendet von Outlook für Android <https://aka.ms/AAb9ysg>
> ------------------------------------------------------------------------
> *From:* Mattias Engdegård <mattiasengdegard <at> gmail.com> on behalf of 
> Mattias Engdegård <mattias.engdegard <at> gmail.com>
> *Sent:* Saturday, April 1, 2023 2:01:19 PM
> *To:* reto <at> gnu.org <reto <at> gnu.org>
> *Cc:* Cyril Arnould <cyril.arnould <at> outlook.com>; Eli Zaretskii 
> <eliz <at> gnu.org>; friedrich.beckmann <at> gmx.de <friedrich.beckmann <at> gmx.de>; 
> 62508 <at> debbugs.gnu.org <62508 <at> debbugs.gnu.org>
> *Subject:* bug#62508: 28.2; vhdl-mode ModelSim compile warnings aren't 
> highlighted
> The new regexp,
>
> "^\\(ERROR\\|WARNING\\|\\*\\* Error\\|\\*\\* Warning\\)[^:]*:\\( 
> *\[[0-9]\]\\| ([^)]+)\\)? \\([^ \t\n]+\\)(\\([0-9]+\\)):"
> ^^^^^^^^^
>
> isn't quite right: there are single-escaped square brackets, and the 
> original regexp permitted multiple digits, not just one, between 
> square brackets. The part
>
>   "\[[0-9]\]"
>
> should probably be
>
>   "\\[[0-9]+]"
>
> but the author needs to confirm this.
>
[Message part 2 (text/html, inline)]

This bug report was last modified 2 years and 49 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.