[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: master d08c947: Make compilation-mode regexp matching case-sensitive

From: Mattias Engdegård
Subject: Re: master d08c947: Make compilation-mode regexp matching case-sensitive (bug#40119)
Date: Wed, 25 Mar 2020 23:29:50 +0100

25 mars 2020 kl. 21.54 skrev Stefan Monnier <address@hidden>:

> The regular expressions are written either with or without case-folding
> in mind, so changing it globally is a recipe for problems.

Agreed, but I doubt that anyone will ever set the option. It was added just in 
case someone has a tool emitting messages that happens to match one of the 
built-in rules if case-folding is used, or if a user has added a sloppily 
written regexp. In either case, adding a properly written regexp is of course 
the recommended action.

> If we want to allow case-insensitive regexp matching, then we should
> extend the syntax of `compilation-error-regexp-alist` so that a given
> entry there can request it to be used with case-folding.

It would be more flexible, but I'd rather not add the complexity without 
evidence of a need for it.

> PS: The same applies to the "zero/one-based column numbers" and
>    the "char-colums vs visual-columns vs byte-columns", admittedly.

Yes, those settings would make more sense if they were rule-specific. Still, 
actual examples of tools needing them would be useful.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]