emacs-elpa-diffs
[Top][All Lists]
Advanced

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

[nongnu] elpa/rainbow-delimiters f5679e9040 120/188: Simplify two ifs by


From: ELPA Syncer
Subject: [nongnu] elpa/rainbow-delimiters f5679e9040 120/188: Simplify two ifs by using max in their stead.
Date: Sat, 1 Jan 2022 00:58:58 -0500 (EST)

branch: elpa/rainbow-delimiters
commit f5679e904033e0b5dcac0eb3160e324213cd2cd1
Author: Fanael Linithien <fanael4@gmail.com>
Commit: Fanael Linithien <fanael4@gmail.com>

    Simplify two ifs by using max in their stead.
---
 rainbow-delimiters.el | 18 +++++-------------
 1 file changed, 5 insertions(+), 13 deletions(-)

diff --git a/rainbow-delimiters.el b/rainbow-delimiters.el
index a223516677..efaad948d8 100644
--- a/rainbow-delimiters.el
+++ b/rainbow-delimiters.el
@@ -287,15 +287,6 @@ to work around a bug."
             (rainbow-delimiters--syntax-ppss-run (car nearest-before) pos (cdr 
nearest-before))
           (rainbow-delimiters--syntax-ppss-run (point-min) pos nil))))))
 
-;;; Nesting level
-
-(defun rainbow-delimiters--depth (ppss)
-  "Return # of nested levels of delimiters at parse state PPSS."
-  (let ((depth (car ppss)))
-    (if (>= depth 0)
-        depth
-      0))) ; Ignore negative depths created by unmatched closing parens.
-
 ;;; Text properties
 
 (defun rainbow-delimiters--propertize-delimiter (loc depth match)
@@ -391,7 +382,8 @@ Used by font-lock for dynamic highlighting."
     ;; Point can be anywhere in buffer; determine the nesting depth at point.
     (let* ((last-ppss-pos (point))
            (ppss (rainbow-delimiters--syntax-ppss last-ppss-pos))
-           (depth (rainbow-delimiters--depth ppss)))
+           ;; Ignore negative depths created by unmatched closing delimiters.
+           (depth (max 0 (nth 0 ppss))))
       (while (and (< (point) end)
                   (re-search-forward rainbow-delimiters--delim-regex end t))
         (let* ((delim-pos (match-beginning 0))
@@ -412,9 +404,9 @@ Used by font-lock for dynamic highlighting."
                                                  delim-pos
                                                  (eq (cdr delim-syntax)
                                                      matching-opening-delim))
-                (setq depth (if (<= depth 0)
-                                0 ; unmatched delim
-                              (1- depth))))))))))
+                ;; Don't let `depth' go negative, even if there's an unmatched
+                ;; delimiter.
+                (setq depth (max 0 (1- depth))))))))))
   ;; We already fontified the delimiters, tell font-lock there's nothing more
   ;; to do.
   nil)



reply via email to

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