]> git.eshelyaron.com Git - emacs.git/commitdiff
Fix one more 2019-08-04 regex lint
authorPaul Eggert <eggert@cs.ucla.edu>
Sun, 4 Aug 2019 22:54:17 +0000 (15:54 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Sun, 4 Aug 2019 22:54:56 +0000 (15:54 -0700)
Problem clarified by Mattias EngdegĂ„rd in:
https://lists.gnu.org/r/emacs-devel/2019-08/msg00087.html
* lisp/isearch.el (isearch-symbol-regexp):
Remove \s@ from regexp as it cannot match.

lisp/isearch.el

index 97c75b2978b5be07f515ddaca70f542f35e1c82b..09729034d748171f87eb58627d2d6644e5773a68 100644 (file)
@@ -2184,16 +2184,19 @@ matches arbitrary non-symbol whitespace.  Otherwise if LAX is non-nil,
 the beginning or the end of the string need not match a symbol boundary."
   (let ((not-word-symbol-re
         ;; This regexp matches all syntaxes except word and symbol syntax.
-        ;; FIXME: Replace it with something shorter if possible (bug#14602).
-        "\\(?:\\s-\\|\\s.\\|\\s(\\|\\s)\\|\\s\"\\|\\s\\\\|\\s/\\|\\s$\\|\\s'\\|\\s<\\|\\s>\\|\\s@\\|\\s!\\|\\s|\\)+"))
+        "\\(?:\\s-\\|\\s.\\|\\s(\\|\\s)\\|\\s\"\\|\\s\\\\|\\s/\\|\\s$\\|\\s'\\|\\s<\\|\\s>\\|\\s!\\|\\s|\\)+"))
     (cond
      ((equal string "") "")
-     ((string-match-p (format "\\`%s\\'" not-word-symbol-re) string) not-word-symbol-re)
+     ((string-match-p (format "\\`%s\\'" not-word-symbol-re) string)
+      not-word-symbol-re)
      (t (concat
-        (if (string-match-p (format "\\`%s" not-word-symbol-re) string) not-word-symbol-re
+        (if (string-match-p (format "\\`%s" not-word-symbol-re) string)
+            not-word-symbol-re
           "\\_<")
-        (mapconcat 'regexp-quote (split-string string not-word-symbol-re t) not-word-symbol-re)
-        (if (string-match-p (format "%s\\'" not-word-symbol-re) string) not-word-symbol-re
+        (mapconcat 'regexp-quote (split-string string not-word-symbol-re t)
+                   not-word-symbol-re)
+        (if (string-match-p (format "%s\\'" not-word-symbol-re) string)
+            not-word-symbol-re
           (unless lax "\\_>")))))))
 
 ;; Search with lax whitespace