From d94dc606a0934e52f86bd939684867ada4b944fe Mon Sep 17 00:00:00 2001 From: Dmitry Gutov Date: Fri, 20 Jan 2023 05:35:12 +0200 Subject: [PATCH] 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 -- 2.39.2