[ncalexan@math.uci.edu: Issue with rx: (rx (and anything anything)) seems incorrect]

[ncalexan@math.uci.edu: Issue with rx: (rx (and anything anything)) seems incorrect]

[Richard Stallman]

Would someone please DTRT, then ack?

------- Start of forwarded message -------
X-Spam-Status: No, score=0.0 required=5.0 tests=UNPARSEABLE_RELAY 
	autolearn=failed version=3.1.0
To: emacs-pretest-bug@gnu.org
From: Nick Alexander <ncalexan@math.uci.edu>
Date: Fri, 29 Jun 2007 11:48:41 -0700
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: Issue with rx: (rx (and anything anything)) seems incorrect

In GNU Emacs 22.0.95.1 (powerpc-apple-darwin8.9.0, Carbon Version 1.6.0)
 of 2007-03-15 on g5.tokyo.stp.isas.jaxa.jp
Windowing system distributor `Apple Inc.', version 10.4.9
configured using `configure  '--prefix=/Applications/Emacs.app/Contents/Resources' '--with-carbon' '--without-x' '--libexecdir=/Volumes/Emacs/Emacs.app/Contents/MacOS/libexec' 'CFLAGS=-arch i386 -arch ppc -isysroot /Developer/SDKs/MacOSX10.4u.sdk -DUSE_ATSUI -DUSE_MAC_TSM''

Hi all,

The following is expected:
(rx (and ?a ?b)) => "\\(?:ab\\)"

The following is not:
(rx (and anything anything)) => "\\(?:.\\|
.\\|
\\)"

I have no idea why the newlines are inserted, nor why there are choice
bars (|).  Why not just "\\(?:..\\)" as before?

Nick Alexander


_______________________________________________
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
------- End of forwarded message -------

Re: Fwd: Issue with rx: (rx (and anything anything)) seems incorrect

[Stefan Monnier]

> Would someone please DTRT, then ack?

Fixed,

> The following is not:
> (rx (and anything anything)) => "\\(?:.\\|
> .\\|
> \\)"

> I have no idea why the newlines are inserted, nor why there are choice
> bars (|).  Why not just "\\(?:..\\)" as before?

anything matches any char, whereas . does not match newlines.  But the
result you see is indeed wrong.  It should be
"\\(?:.\\|\n\\)\\(?:.\\|\n\\)".  Thanks for the report,


        Stefan