GNU bug report logs - #63251
28.2; vhdl-mode contribution

Previous Next

Package: emacs;

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

Date: Wed, 3 May 2023 19:46: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 #34 received at 63251 <at> debbugs.gnu.org (full text, mbox):

From: Cyril Arnould <cyril.arnould <at> outlook.com>
To: Mattias Engdegård <mattias.engdegard <at> gmail.com>
Cc: Reto Zimmermann <reto <at> gnu.org>, Eli Zaretskii <eliz <at> gnu.org>,
 "63251 <at> debbugs.gnu.org" <63251 <at> debbugs.gnu.org>
Subject: AW: bug#63251: 28.2; vhdl-mode contribution
Date: Sun, 7 May 2023 15:56:45 +0000

[Message part 1 (text/plain, inline)]
Very sorry, I just noticed that I messed up the previous patch. Third time’s the charm.

________________________________
Von: Mattias Engdegård <mattiasengdegard <at> gmail.com> im Auftrag von Mattias Engdegård <mattias.engdegard <at> gmail.com>
Gesendet: Sunday, May 7, 2023 10:17:03 AM
An: Cyril Arnould <cyril.arnould <at> outlook.com>
Cc: 63251 <at> debbugs.gnu.org <63251 <at> debbugs.gnu.org>; Reto Zimmermann <reto <at> gnu.org>; Eli Zaretskii <eliz <at> gnu.org>
Betreff: Re: bug#63251: 28.2; vhdl-mode contribution

The vhdl-mode maintainers need to look at your patch more closely; I just have some minor remarks.

7 maj 2023 kl. 00.11 skrev Cyril Arnould <cyril.arnould <at> outlook.com>:

> - I've added TYPE to the vhdl-compiler definition with the
>   appropriate choices for Info/Warning/Error and the dotted
>   pair. I'm not sure if sexp was the correct choice for the
>   dotted pair, is there a better alternative?

There is (cons ...) which would be more precise, see the manual.

The new doc string says that a TYPE of 2 is allowed but the type spec doesn't allow it.
Either allow both 2 and nil or change the docs to only mention one of them.

> - I added another entry to the backwards compatibility code, all
>   it took was a slight modification of the entry before
>   that.

That's fine, but I'd be a bit more careful with the destructive in-place changes and quoted list constants. (Think of what happens if later code performs an in-place change of that nil you added.)
This isn't performance-critical-code, we can afford consing here.


[Message part 2 (text/html, inline)]
[fix-vhdl-compiler-customization-3.patch (application/octet-stream, attachment)]
This bug report was last modified 2 years and 15 days ago.

Previous Next

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