emacs-diffs
[Top][All Lists]
Advanced

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

emacs-29 d94dc606a09 1/2: ruby-ts-mode: Claw back half of the performanc


From: Dmitry Gutov
Subject: emacs-29 d94dc606a09 1/2: ruby-ts-mode: Claw back half of the performance drop from last change
Date: Thu, 19 Jan 2023 22:42:41 -0500 (EST)

branch: emacs-29
commit d94dc606a0934e52f86bd939684867ada4b944fe
Author: Dmitry Gutov <dgutov@yandex.ru>
Commit: Dmitry Gutov <dgutov@yandex.ru>

    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.
---
 lisp/progmodes/ruby-ts-mode.el | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

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



reply via email to

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