[Emacs-diffs] master 45ee24e: Allow floats as 'pcase' QPATS

[Michael Heerdegen]

branch: master
commit 45ee24efed57093b421159ca1028097952f2d564
Author: Michael Heerdegen <address@hidden>
Commit: Michael Heerdegen <address@hidden>

    Allow floats as 'pcase' QPATS
    
    * lisp/emacs-lisp/pcase.el (\`): Extend semantics of QPATS to all
    numbers.  Add a comment explaining why we disallow some atoms as
    QPATS.
    * doc/lispref/control.texi (Backquote Patterns): Update the paragraph
    explaining QPATS.  Remove a sentence suggesting an analogy between
    QPATS to self-quoting objects.
---
 doc/lispref/control.texi | 4 +---
 lisp/emacs-lisp/pcase.el | 7 +++++--
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/doc/lispref/control.texi b/doc/lispref/control.texi
index 34f5f57..975ab3d 100644
--- a/doc/lispref/control.texi
+++ b/doc/lispref/control.texi
@@ -1090,12 +1090,10 @@ Matches if @var{expval} is a vector of length @var{m} 
whose
 
 @item @var{symbol}
 @itemx @var{keyword}
address@hidden @var{integer}
address@hidden @var{number}
 @itemx @var{string}
 Matches if the corresponding element of @var{expval} is
 @code{equal} to the specified literal object.
-Note that, aside from @var{symbol}, this is the same set of
-self-quoting literal objects that are acceptable as a core pattern.
 
 @item ,@var{pattern}
 Matches if the corresponding element of @var{expval}
diff --git a/lisp/emacs-lisp/pcase.el b/lisp/emacs-lisp/pcase.el
index fa7b1de..4a69244 100644
--- a/lisp/emacs-lisp/pcase.el
+++ b/lisp/emacs-lisp/pcase.el
@@ -919,7 +919,7 @@ QPAT can take the following forms:
   ,PAT                  matches if the `pcase' pattern PAT matches.
   SYMBOL                matches if EXPVAL is `equal' to SYMBOL.
   KEYWORD               likewise for KEYWORD.
-  INTEGER               likewise for INTEGER.
+  NUMBER                likewise for NUMBER.
   STRING                likewise for STRING.
 
 The list or vector QPAT is a template.  The predicate formed
@@ -949,7 +949,10 @@ The predicate is the logical-AND of:
     `(and (pred consp)
           (app car ,(list '\` (car qpat)))
           (app cdr ,(list '\` (cdr qpat)))))
-   ((or (stringp qpat) (integerp qpat) (symbolp qpat)) `',qpat)
+   ((or (stringp qpat) (numberp qpat) (symbolp qpat)) `',qpat)
+   ;; In all other cases just raise an error so we can't break
+   ;; backward compatibility when adding \` support for other
+   ;; compounded values that are not `consp'
    (t (error "Unknown QPAT: %S" qpat))))
 
 (provide 'pcase)