From: Cameron Desautels <camdez@gmail.com>
To: 16046@debbugs.gnu.org
Subject: bug#16046: Bug with Regexp Containing only a Character Class with a Caret
Date: Tue, 3 Dec 2013 22:57:56 -0600 [thread overview]
Message-ID: <CALtQdcmC2QXOGBRKVJq2r=Z-GpAzct+8UbZgpKYcbYK9R2Fd8w@mail.gmail.com> (raw)
Hi all,
I've run across a dilemma, in the most literal sense: either there's a
problem in Emacs's regexp engine or there's an issue with
`regexp-opt-charset`---I'm not sure which.
The issue has to do with regular expressions containing character
classes with only a caret character. I know this seems like a rather
silly case (why not just use "\\^"?) but it came up in the context of
trying to track down a bug in ruby-mode, so it does occur in real (and
particularly *programmatic*) settings.
The simplest case to reproduce is the following:
(re-search-forward "[^]")
; => Debugger entered--Lisp error: (invalid-regexp "Unmatched [ or [^")
; re-search-forward("[^]")
; eval((re-search-forward "[^]") nil)
; eval-last-sexp-1(t)
; eval-last-sexp(t)
; eval-print-last-sexp()
; call-interactively(eval-print-last-sexp record nil)
; command-execute(eval-print-last-sexp record)
; execute-extended-command(nil "eval-print-last-sexp")
; call-interactively(execute-extended-command nil nil)
Now, you can make a compelling case that that's not a valid regexp
(and the Emacs Lisp Reference Manual doesn't seem to *directly*
contradict this argument), but that presents a problem when paired
with `regexp-opt-charset`:
(regexp-opt-charset '(?^))
=> "[^]"
Note that that produces the problem regexp; which is to say that the
following code is bound to fail when it should succeed:
(re-search-forward (regexp-opt-charset '(?^)))
What's the correct behavior? I'd be happy to offer a patch for either
side of the equation but I'm not sure which one to target.
All the best.
-- Cameron
In GNU Emacs 24.3.1 (x86_64-apple-darwin11.4.2, Carbon Version 1.6.0
AppKit 1138.51)
of 2013-05-13 on atago
Windowing system distributor `Apple Inc.', version 10.9.0
Configured using:
`configure '--with-mac'
'--enable-mac-app=/Users/xin/Documents/emacs-mac-port/build'
'--prefix=/Users/xin/Documents/emacs-mac-port/build''
Important settings:
value of $LANG: en_US.UTF-8
locale-coding-system: utf-8-unix
default enable-multibyte-characters: t
Major mode: Lisp Interaction
Minor modes in effect:
tooltip-mode: t
mouse-wheel-mode: t
tool-bar-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
line-number-mode: t
transient-mark-mode: t
Load-path shadows:
/Applications/Emacs.app/Contents/Resources/lisp/.dir-locals hides
/Applications/Emacs.app/Contents/Resources/lisp/gnus/.dir-locals
Features:
(shadow sort gnus-util mail-extr emacsbug message format-spec rfc822 mml
mml-sec mm-decode mm-bodies mm-encode mail-parse rfc2231 mailabbrev
gmm-utils mailheader sendmail rfc2047 rfc2045 ietf-drums mm-util
mail-prsvr mail-utils help-mode easymenu debug time-date tooltip
ediff-hook vc-hooks lisp-float-type mwheel mac-win tool-bar dnd fontset
image regexp-opt fringe tabulated-list newcomment lisp-mode register
page menu-bar rfn-eshadow timer select scroll-bar mouse jit-lock
font-lock syntax facemenu font-core frame cham georgian utf-8-lang
misc-lang vietnamese tibetan thai tai-viet lao korean japanese hebrew
greek romanian slovak czech european ethiopic indian cyrillic chinese
case-table epa-hook jka-cmpr-hook help simple abbrev minibuffer loaddefs
button faces cus-face macroexp files text-properties overlay sha1 md5
base64 format env code-pages mule custom widget hashtable-print-readable
backquote mac multi-tty make-network-process emacs)
next reply other threads:[~2013-12-04 4:57 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-04 4:57 Cameron Desautels [this message]
2013-12-05 19:26 ` bug#16046: Bug with Regexp Containing only a Character Class with a Caret (PATCH) Cameron Desautels
2013-12-05 20:26 ` Stefan Monnier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CALtQdcmC2QXOGBRKVJq2r=Z-GpAzct+8UbZgpKYcbYK9R2Fd8w@mail.gmail.com' \
--to=camdez@gmail.com \
--cc=16046@debbugs.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.