From: Dmitry Gutov <dgutov@yandex.ru>
Date: Fri, 20 Jan 2023 03:35:12 +0000 (+0200)
Subject: ruby-ts-mode: Claw back half of the performance drop from last change
X-Git-Tag: emacs-29.0.90~652
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=d94dc606a0934e52f86bd939684867ada4b944fe;p=emacs.git

ruby-ts-mode: Claw back half of the performance drop from last change

* lisp/progmodes/ruby-ts-mode.el (ruby-ts--builtin-method-p):
New function.
(ruby-ts--font-lock-settings): Use it instead of :match.
---

diff --git a/lisp/progmodes/ruby-ts-mode.el b/lisp/progmodes/ruby-ts-mode.el
index 58da5ef9c69..2105aaaecab 100644
--- a/lisp/progmodes/ruby-ts-mode.el
+++ b/lisp/progmodes/ruby-ts-mode.el
@@ -208,6 +208,9 @@ values of OVERRIDE"
     (treesit-fontify-with-override (max plus-1 start) (min node-end end)
                                    font-lock-comment-face override)))
 
+(defun ruby-ts--builtin-method-p (node)
+  (string-match-p ruby-ts--builtin-methods (treesit-node-text node t)))
+
 (defun ruby-ts--font-lock-settings (language)
   "Tree-sitter font-lock settings for Ruby."
   (treesit-font-lock-rules
@@ -336,8 +339,7 @@ values of OVERRIDE"
    :language language
    :feature 'builtin-function
    `((((identifier) @font-lock-builtin-face)
-      (:match ,ruby-ts--builtin-methods
-       @font-lock-builtin-face)))
+      (:pred ruby-ts--builtin-method-p @font-lock-builtin-face)))
 
    ;; Yuan recommends also putting method definitions into the
    ;; 'function' category (thus keeping it in both).  I've opted to