emacs-diffs
[Top][All Lists]
Advanced

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

master d096e12: Simplify `gnu` compilation-mode regexp


From: Mattias Engdegård
Subject: master d096e12: Simplify `gnu` compilation-mode regexp
Date: Mon, 22 Nov 2021 13:12:25 -0500 (EST)

branch: master
commit d096e12f447c1c67fe6fb6baa44212781d27ef53
Author: Mattias Engdegård <mattiase@acm.org>
Commit: Mattias Engdegård <mattiase@acm.org>

    Simplify `gnu` compilation-mode regexp
    
    * lisp/progmodes/compile.el (compilation-error-regexp-alist-alist):
    Remove the pattern ostensibly added for Ruby, because at closer
    inspection it could never have matched anything.  This lessens the
    performance impact of the pattern added for GCC's -fanalyzer,
    now slightly tweaked.
---
 lisp/progmodes/compile.el | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/lisp/progmodes/compile.el b/lisp/progmodes/compile.el
index c0e16ce..2d4070c 100644
--- a/lisp/progmodes/compile.el
+++ b/lisp/progmodes/compile.el
@@ -346,15 +346,10 @@ of[ \t]+\"?\\([a-zA-Z]?:?[^\":\n]+\\)\"?:" 3 2 nil (1))
        ;;     PROGRAM:SOURCE-FILE-NAME:LINENO: MESSAGE
        ;; which is used for non-interactive programs other than
        ;; compilers (e.g. the "jade:" entry in compilation.txt).
-       (? (| (regexp "[[:alpha:]][-[:alnum:].]+: ?")
-             ;; FIXME: This pattern was added for handling messages
-             ;; from Ruby, but it is unclear whether it is actually
-             ;; used since the gcc-include rule above seems to cover
-             ;; it.
-             (regexp "[ \t]+\\(?:in \\|from\\)")
+       (? (| (: alpha (+ (in ?. ?- alnum)) ":" (? " "))
              ;; Skip indentation generated by tools like GCC's
              ;; -fanalyzer.
-             (: (+ space) "|")))
+             (: (+ (in " \t")) "|")))
 
        ;; File name group.
        (group-n 1



reply via email to

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