GNU bug report logs - #60830
30.0.50; The *Compilation* buffer does not recognize Lua errors

Previous Next

Package: emacs;

Reported by: Rudolf Adamkovič <salutis <at> me.com>

Date: Sun, 15 Jan 2023 11:35:01 UTC

Severity: wishlist

Found in version 30.0.50

Done: Stefan Kangas <stefankangas <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Mattias Engdegård <mattias.engdegard <at> gmail.com>
To: Rudolf Adamkovič <salutis <at> me.com>
Cc: 60830 <at> debbugs.gnu.org, Stefan Kangas <stefankangas <at> gmail.com>, Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: bug#60830: 30.0.50; The *Compilation* buffer does not recognize Lua errors
Date: Sun, 8 Oct 2023 11:59:18 +0200
7 okt. 2023 kl. 23.02 skrev Rudolf Adamkovič <salutis <at> me.com>:

> (1) How can I specify "this group cannot be '[C]'" in the regexp?

Disallow files starting with a left square bracket?

> (2) How can I test this scenario in 'compile-tests.el'?

We can help out.

> I made the relevant groups greedy, but I wonder what difference it
> makes.

Only use non-greedy operators if they make the regexp significantly easier to express. Greedy should be the default.
Non-greedy operators are usually less efficient, and always raise eyebrows when read: why is this non-greedy?

Writing ".*?\n", as you did, makes no sense at all since "." will match anything except newline, and the result is identical to ".*\n" but less readable and maybe slower.

>> We try to make rules work with Windows file names (or names containing
>> spaces, which are also somewhat common on Windows) where relevant and
>> practical. Your patterns don't; can you argue that it's not useful for
>> Lua or not possible to do so?
> 
> Good catch!  Fixed.

Maybe you should test it just in case. (You don't need a Windows machine to do that but it adds authenticity).

> I would understand if we did that for some long-dead compilers but not
> for Lua.

Look, it's not a beauty contest. We'll do what we have to.

> We still detect the file path/name, which is useful.

The point is that it substantially increases the risks of false matches. Lua needs to be a good neighbour and can't just think of its own needs over anything else.





This bug report was last modified 1 year and 211 days ago.

Previous Next


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