diff options
| author | Mattias Engdegård <mattiase@acm.org> | 2021-01-05 12:01:32 +0100 |
|---|---|---|
| committer | Mattias Engdegård <mattiase@acm.org> | 2021-01-05 12:01:32 +0100 |
| commit | 1433a1201447f6f8b610f4d7f78a4b8a739c6572 (patch) | |
| tree | 3bbf04838095912f1b31e8450b9d8d0534f36916 /lisp/progmodes/ruby-mode.el | |
| parent | 77796eb0142f5d9b616f59f271ef380f099f1cac (diff) | |
| download | emacs-1433a1201447f6f8b610f4d7f78a4b8a739c6572.tar.gz emacs-1433a1201447f6f8b610f4d7f78a4b8a739c6572.tar.bz2 emacs-1433a1201447f6f8b610f4d7f78a4b8a739c6572.zip | |
ruby-mode: eliminate redundant regexp branch
* lisp/progmodes/ruby-mode.el (ruby-add-log-current-method):
Since ruby-operator-re matches dot, don't include both in regexp.
This pacifies relint.
Diffstat (limited to 'lisp/progmodes/ruby-mode.el')
| -rw-r--r-- | lisp/progmodes/ruby-mode.el | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lisp/progmodes/ruby-mode.el b/lisp/progmodes/ruby-mode.el index cd9d087856c..a8667acb9d5 100644 --- a/lisp/progmodes/ruby-mode.el +++ b/lisp/progmodes/ruby-mode.el @@ -1603,9 +1603,9 @@ See `add-log-current-defun-function'." "\\(" ;; \\. and :: for class methods "\\([A-Za-z_]" ruby-symbol-re "*[?!]?" - (when method-name? "\\|") - (when method-name? ruby-operator-re) - "\\|\\.\\|::" "\\)" + "\\|" + (if method-name? ruby-operator-re "\\.") + "\\|::" "\\)" "+\\)"))) (definition-re (funcall make-definition-re ruby-defun-beg-re t)) (module-re (funcall make-definition-re "\\(class\\|module\\)"))) |