[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: w3m-local-find-file-regexps parens
- From: Katsumi Yamaoka <yamaoka@xxxxxxx>
- Date: Mon, 20 Apr 2009 13:40:57 +0900
- X-ml-name: emacs-w3m
- X-mail-count: 10832
- References: <871vrqsx44.fsf@xxxxxxxxx>
>>>>> In [emacs-w3m : No.10830] Kevin Ryde wrote:
> In the current cvs w3m-local-find-file-regexps ends up with
> "\\.html?\\|shtml?\\|txt\\|xhtml?\\'"
> but I wonder if there's meant to be some parens around the alternatives
> so the \\. is required for all of them, as shown in the docstring, and
> for example so that the following doesn't match (whereas it currently
> does)
> (string-match (cdr w3m-local-find-file-regexps) "etxt")
Thank you for the patch. I've applied it in CVS. That is due
to a bug of `regexp-opt' that has been fixed in Emacs 23:
2008-05-23 Stefan Monnier <monnier@xxxxxxxxxxxxxxxx>
* emacs-lisp/regexp-opt.el (regexp-opt): Always return
a properly-grouped regexp.
But I verified your patch is still necessary for running w3m.el
with Emacs 21, 22, and XEmacs.
Regards,