[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] lisp/files.el: Add `file-name-set-extension`
|
From: |
Colin Woodbury |
|
Subject: |
Re: [PATCH] lisp/files.el: Add `file-name-set-extension` |
|
Date: |
Tue, 25 May 2021 12:45:58 -0700 |
|
User-agent: |
Cyrus-JMAP/3.5.0-alpha0-448-gae190416c7-fm-20210505.004-gae190416 |
Thank you, you're right. I've tested the old and revised code, and the double backslash wasn't neccesary.
I've attached the revised patch.
On Tue, 25 May 2021, at 11:00, Basil L. Contovounesios wrote:
> + (let* ((patt "[ \\t\\n\\r.]+") ; Borrowed from `string-trim'.
Those backslashes need escaping only in string-trim's docstring, not in
the literal regexp string.
Thanks,
--
Basil
file-name-set-extension.patch
Description: Text Data
- [PATCH] lisp/files.el: Add `file-name-set-extension`, Colin Woodbury, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Andreas Schwab, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Basil L. Contovounesios, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`,
Colin Woodbury <=
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Stefan Monnier, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Colin Woodbury, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Stefan Monnier, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Colin Woodbury, 2021/05/25
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Andreas Schwab, 2021/05/26
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Colin Woodbury, 2021/05/26
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Andreas Schwab, 2021/05/26
- Re: [PATCH] lisp/files.el: Add `file-name-set-extension`, Colin Woodbury, 2021/05/30