]> git.eshelyaron.com Git - emacs.git/commitdiff
Use `eql` or `eq` instead of `=` in some places
authorMattias Engdegård <mattiase@acm.org>
Sat, 27 Aug 2022 12:20:38 +0000 (14:20 +0200)
committerMattias Engdegård <mattiase@acm.org>
Thu, 15 Sep 2022 07:42:53 +0000 (09:42 +0200)
For a switch op to be generated, comparisons must be made using `eq`,
`eql` or `equal`, not `=`.

* lisp/emacs-lisp/byte-opt.el (byte-optimize-lapcode):
* lisp/files.el (file-modes-char-to-who, file-modes-char-to-right):
* lisp/international/titdic-cnv.el (tit-process-header):
* lisp/language/ethio-util.el (ethio-input-special-character)
(ethio-fidel-to-tex-buffer):
* lisp/language/lao.el (consonant):
Use `eq` or `eql` instead of `=`.

In these cases either `eq` or `eql` would do and the choice does not
affect the resulting code.  We compare numbers with `eql` and
characters with `eq` as a matter of style.

lisp/emacs-lisp/byte-opt.el
lisp/files.el
lisp/international/titdic-cnv.el
lisp/language/ethio-util.el
lisp/language/lao.el

index 27b0d33d3ef13f4ea7bd2589a3d8fc33b0ecd6ef..0d5f8c26eb2d0999a91909ab649ddcb6cbe94ba1 100644 (file)
@@ -1999,20 +1999,20 @@ If FOR-EFFECT is non-nil, the return value is assumed to be of no importance."
          (setq keep-going t)
          (setq tmp (aref byte-stack+-info (symbol-value (car lap0))))
          (setq rest (cdr rest))
-         (cond ((= tmp 1)
+         (cond ((eql tmp 1)
                 (byte-compile-log-lap
                  "  %s discard\t-->\t<deleted>" lap0)
                 (setq lap (delq lap0 (delq lap1 lap))))
-               ((= tmp 0)
+               ((eql tmp 0)
                 (byte-compile-log-lap
                  "  %s discard\t-->\t<deleted> discard" lap0)
                 (setq lap (delq lap0 lap)))
-               ((= tmp -1)
+               ((eql tmp -1)
                 (byte-compile-log-lap
                  "  %s discard\t-->\tdiscard discard" lap0)
                 (setcar lap0 'byte-discard)
                 (setcdr lap0 0))
-               ((error "Optimizer error: too much on the stack"))))
+               ((error "Optimizer error: too much on the stack"))))
         ;;
         ;; goto*-X X:  -->  X:
         ;;
index 540bc2a6a85ee50a1556e06857855fe31597ea7b..0f2d3ca4b9a7222f1420b3a19c9571b7b4bd94d4 100644 (file)
@@ -8271,10 +8271,10 @@ CHAR is in [ugoa] and represents the category of users (Owner, Group,
 Others, or All) for whom to produce the mask.
 The bit-mask that is returned extracts from mode bits the access rights
 for the specified category of users."
-  (cond ((= char ?u) #o4700)
-       ((= char ?g) #o2070)
-       ((= char ?o) #o1007)
-       ((= char ?a) #o7777)
+  (cond ((eq char ?u) #o4700)
+       ((eq char ?g) #o2070)
+       ((eq char ?o) #o1007)
+       ((eq char ?a) #o7777)
         (t (error "%c: Bad `who' character" char))))
 
 (defun file-modes-char-to-right (char &optional from)
@@ -8282,22 +8282,22 @@ for the specified category of users."
 CHAR is in [rwxXstugo] and represents symbolic access permissions.
 If CHAR is in [Xugo], the value is taken from FROM (or 0 if omitted)."
   (or from (setq from 0))
-  (cond ((= char ?r) #o0444)
-       ((= char ?w) #o0222)
-       ((= char ?x) #o0111)
-       ((= char ?s) #o6000)
-       ((= char ?t) #o1000)
+  (cond ((eq char ?r) #o0444)
+       ((eq char ?w) #o0222)
+       ((eq char ?x) #o0111)
+       ((eq char ?s) #o6000)
+       ((eq char ?t) #o1000)
        ;; Rights relative to the previous file modes.
-       ((= char ?X) (if (= (logand from #o111) 0) 0 #o0111))
-       ((= char ?u) (let ((uright (logand #o4700 from)))
-                      ;; FIXME: These divisions/shifts seem to be right
-                       ;; for the `7' part of the #o4700 mask, but not
-                       ;; for the `4' part.  Same below for `g' and `o'.
-                      (+ uright (/ uright #o10) (/ uright #o100))))
-       ((= char ?g) (let ((gright (logand #o2070 from)))
-                      (+ gright (/ gright #o10) (* gright #o10))))
-       ((= char ?o) (let ((oright (logand #o1007 from)))
-                      (+ oright (* oright #o10) (* oright #o100))))
+       ((eq char ?X) (if (= (logand from #o111) 0) 0 #o0111))
+       ((eq char ?u) (let ((uright (logand #o4700 from)))
+                       ;; FIXME: These divisions/shifts seem to be right
+                        ;; for the `7' part of the #o4700 mask, but not
+                        ;; for the `4' part.  Same below for `g' and `o'.
+                       (+ uright (/ uright #o10) (/ uright #o100))))
+       ((eq char ?g) (let ((gright (logand #o2070 from)))
+                       (+ gright (/ gright #o10) (* gright #o10))))
+       ((eq char ?o) (let ((oright (logand #o1007 from)))
+                       (+ oright (* oright #o10) (* oright #o100))))
         (t (error "%c: Bad right character" char))))
 
 (defun file-modes-rights-to-number (rights who-mask &optional from)
index 080045e7520fcad77fe42e24609ae4f4a10f87a7..d2a6ee1e9d125ad679e9c6a162ecb4a847f17097 100644 (file)
@@ -281,7 +281,7 @@ SPC, 6, 3, 4, or 7 specifying a tone (SPC:陰平, 6:陽平, 3:上聲, 4:去聲,
     (while (not (eobp))
       (let ((ch (following-char))
            (pos (point)))
-       (cond ((= ch ?C)                ; COMMENT
+       (cond ((eq ch ?C)               ; COMMENT
               (cond ((looking-at "COMMENT")
                      (let ((pos (match-end 0))
                            (to (progn (end-of-line) (point))))
@@ -295,7 +295,7 @@ SPC, 6, 3, 4, or 7 specifying a tone (SPC:陰平, 6:陽平, 3:上聲, 4:去聲,
                        (setq tit-comments
                              (cons (buffer-substring-no-properties pos (point))
                                    tit-comments))))))
-             ((= ch ?M)                ; MULTICHOICE, MOVERIGHT, MOVELEFT
+             ((eq ch ?M)               ; MULTICHOICE, MOVERIGHT, MOVELEFT
               (cond ((looking-at "MULTICHOICE:[ \t]*")
                      (goto-char (match-end 0))
                      (setq tit-multichoice (looking-at "YES")))
@@ -305,7 +305,7 @@ SPC, 6, 3, 4, or 7 specifying a tone (SPC:陰平, 6:陽平, 3:上聲, 4:去聲,
                     ((looking-at "MOVELEFT:[ \t]*")
                      (goto-char (match-end 0))
                      (setq tit-moveleft (tit-read-key-value)))))
-             ((= ch ?P)                ; PROMPT
+             ((eq ch ?P)               ; PROMPT
               (cond ((looking-at "PROMPT:[ \t]*")
                      (goto-char (match-end 0))
                      (setq tit-prompt (tit-read-key-value))
@@ -316,7 +316,7 @@ SPC, 6, 3, 4, or 7 specifying a tone (SPC:陰平, 6:陽平, 3:上聲, 4:去聲,
                        (if (or (eq (nth 1 split) 32)
                                (eq (nth 2 split) 32))
                            (setq tit-prompt (substring tit-prompt 0 -1)))))))
-             ((= ch ?B)                ; BACKSPACE, BEGINDICTIONARY,
+             ((eq ch ?B)               ; BACKSPACE, BEGINDICTIONARY,
                                        ; BEGINPHRASE
               (cond ((looking-at "BACKSPACE:[ \t]*")
                      (goto-char (match-end 0))
@@ -325,7 +325,7 @@ SPC, 6, 3, 4, or 7 specifying a tone (SPC:陰平, 6:陽平, 3:上聲, 4:去聲,
                      (setq tit-dictionary t))
                     ((looking-at "BEGINPHRASE")
                      (setq tit-dictionary nil))))
-             ((= ch ?K)                ; KEYPROMPT
+             ((eq ch ?K)               ; KEYPROMPT
               (cond ((looking-at "KEYPROMPT(\\(.*\\)):[ \t]*")
                      (let ((key-char (match-string 1)))
                        (goto-char (match-end 0))
index a0159679da223be0dc82d3d694ffc80709126399..2f76acfe7cbfc361a46aa3d17231cca295e67d65 100644 (file)
@@ -794,15 +794,15 @@ The 2nd and 3rd arguments BEGIN and END specify the region."
   "This function is deprecated."
   (interactive "*cInput number: 1.  2.  3.  4.  5.")
   (cond
-   ((= arg ?1)
+   ((eq arg ?1)
     (insert ""))
-   ((= arg ?2)
+   ((eq arg ?2)
     (insert ""))
-   ((= arg ?3)
+   ((eq arg ?3)
     (insert ""))
-   ((= arg ?4)
+   ((eq arg ?4)
     (insert ""))
-   ((= arg ?5)
+   ((eq arg ?5)
     (insert ""))
    (t
     (error ""))))
@@ -816,7 +816,7 @@ The 2nd and 3rd arguments BEGIN and END specify the region."
   "Convert each fidel characters in the current buffer into a fidel-tex command."
   (interactive)
   (let ((buffer-read-only nil)
-       comp ch)
+       comp)
 
     ;; Special treatment for geminated characters.
     ;; Geminated characters la", etc. change into \geminateG{\laG}, etc.
@@ -835,21 +835,22 @@ The 2nd and 3rd arguments BEGIN and END specify the region."
     ;; Special Ethiopic punctuation.
     (goto-char (point-min))
     (while (re-search-forward "\\ce[».?]\\|«\\ce" nil t)
-      (cond
-       ((= (setq ch (preceding-char)) ?\»)
-       (delete-char -1)
-       (insert "\\rquoteG"))
-       ((= ch ?.)
-       (delete-char -1)
-       (insert "\\dotG"))
-       ((= ch ??)
-       (delete-char -1)
-       (insert "\\qmarkG"))
-       (t
-       (forward-char -1)
-       (delete-char -1)
-       (insert "\\lquoteG")
-       (forward-char 1))))
+      (let ((ch (preceding-char)))
+        (cond
+         ((eq ch ?\»)
+         (delete-char -1)
+         (insert "\\rquoteG"))
+         ((eq ch ?.)
+         (delete-char -1)
+         (insert "\\dotG"))
+         ((eq ch ??)
+         (delete-char -1)
+         (insert "\\qmarkG"))
+         (t
+         (forward-char -1)
+         (delete-char -1)
+         (insert "\\lquoteG")
+         (forward-char 1)))))
 
     ;; Ethiopic characters to TeX macros
     (robin-invert-region (point-min) (point-max) "ethiopic-tex")
index 1861eff15eb8c0c6c0caf1da6aae901d6a1a2f3d..0ad5b9f84e31cc351b03709e62f44c167c6b84c4 100644 (file)
@@ -60,9 +60,9 @@
           (len (length chars))
           ;; Replace `c', `t', `v' to consonant, tone, and vowel.
            (regexp (mapconcat (lambda (c)
-                                (cond ((= c ?c) consonant)
-                                      ((= c ?t) tone)
-                                      ((= c ?v) vowel-upper-lower)
+                                (cond ((eq c ?c) consonant)
+                                      ((eq c ?t) tone)
+                                      ((eq c ?v) vowel-upper-lower)
                                       (t (string c))))
                              (cdr l) ""))
           ;; Element of composition-function-table.