From 2a6e91240f8ff72cc340b7910cec23eea0533ccd Mon Sep 17 00:00:00 2001 From: Kaz Kylheku Date: Sun, 24 Jan 2021 13:10:20 -0800 Subject: matcher: rescind support for @(rcons ...) patterns. There is no longer any way to write a @(rcons ...) pattern using the range syntax, so there is no point in supporting that operator. The silly syntax @@a..@b which previously worked was actually due to a mistaken requirement in the parser. * share/txr/stdlib/match.tl (compile-range-match): Function moved closer to compile-atom-match, below compile-vec-match. The argument is now a range object containing patterns, so we pull it apart with from and to. (compile-atom-match): Pass range directly to compile-range-match; no need to construct (rcons ...) syntax. * tests/011/patmatch.tl: Add range tests from documentation and a few others. * txr.1: References to @(rcons ...) pattern scrubbed. One wrong #R pattern example corrected. --- share/txr/stdlib/match.tl | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-) (limited to 'share') diff --git a/share/txr/stdlib/match.tl b/share/txr/stdlib/match.tl index 8161d57f..d1c45559 100644 --- a/share/txr/stdlib/match.tl +++ b/share/txr/stdlib/match.tl @@ -156,6 +156,24 @@ vars (mappend .vars elem-matches) var-exprs (mappend .var-exprs elem-matches)))) +(defun compile-range-match (rcons-expr obj-var var-list) + (let ((from (from rcons-expr)) + (to (to rcons-expr))) + (let* ((from-match (compile-match from (gensym "from") var-list)) + (to-match (compile-match to (gensym "to") var-list)) + (guard (new match-guard + guard-expr ^(rangep ,obj-var) + pure-vars (list from-match.obj-var to-match.obj-var) + pure-var-exprs (list ^(from ,obj-var) ^(to ,obj-var))))) + (new compiled-match + pattern rcons-expr + obj-var obj-var + guard-chain (cons guard (append from-match.guard-chain + to-match.guard-chain)) + test-expr ^(and ,from-match.test-expr ,to-match.test-expr) + vars (append from-match.vars to-match.vars) + var-exprs (append from-match.var-exprs to-match.var-exprs))))) + (defun compile-atom-match (atom obj-var var-list) (flet ((compile-as-atom () (new compiled-match @@ -167,8 +185,7 @@ (compile-vec-match atom obj-var var-list) (compile-as-atom))) (range (if (non-triv-pat-p atom) - (compile-range-match ^(rcons ,(from atom) ,(to atom)) - obj-var var-list) + (compile-range-match atom obj-var var-list) (compile-as-atom))) (t (compile-as-atom))))) @@ -388,23 +405,6 @@ vars (mappend .vars hash-matches) var-exprs (mappend .var-exprs hash-matches))))) -(defun compile-range-match (rcons-expr obj-var var-list) - (tree-bind (op from to) rcons-expr - (let* ((from-match (compile-match from (gensym "from") var-list)) - (to-match (compile-match to (gensym "to") var-list)) - (guard (new match-guard - guard-expr ^(rangep ,obj-var) - pure-vars (list from-match.obj-var to-match.obj-var) - pure-var-exprs (list ^(from ,obj-var) ^(to ,obj-var))))) - (new compiled-match - pattern rcons-expr - obj-var obj-var - guard-chain (cons guard (append from-match.guard-chain - to-match.guard-chain)) - test-expr ^(and ,from-match.test-expr ,to-match.test-expr) - vars (append from-match.vars to-match.vars) - var-exprs (append from-match.var-exprs to-match.var-exprs))))) - (defun compile-match (pat : (obj-var (gensym)) (var-list (new var-list))) (cond ((consp pat) -- cgit v1.2.3