[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Orgmode] Re: [PATCH] Emacs 22 support
From: |
Bernt Hansen |
Subject: |
[Orgmode] Re: [PATCH] Emacs 22 support |
Date: |
Tue, 01 Jun 2010 23:03:06 -0400 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) |
Štěpán Němec <address@hidden> writes:
> Bernt Hansen <address@hidden> writes:
>
>> * contrib/lisp/org-special-blocks.el: Emacs 22 doesn't have string-match-p
>> ---
>> This patch is available at git://git.norang.ca/org-mode.git emacs22
>>
>> contrib/lisp/org-special-blocks.el | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/contrib/lisp/org-special-blocks.el
>> b/contrib/lisp/org-special-blocks.el
>> index af50b30..4a2bc3c 100644
>> --- a/contrib/lisp/org-special-blocks.el
>> +++ b/contrib/lisp/org-special-blocks.el
>> @@ -51,7 +51,7 @@ seen. This is run after a few special cases are taken care
>> of."
>> (when (or htmlp latexp)
>> (goto-char (point-min))
>> (while (re-search-forward "^#\\+\\(begin\\|end\\)_\\(.*\\)$" nil t)
>> - (unless (string-match-p org-special-blocks-ignore-regexp (match-string 2))
>> + (unless (string-match org-special-blocks-ignore-regexp (match-string 2))
>> (replace-match
>> (if (equal (downcase (match-string 1)) "begin")
>> (concat "ORG-" (match-string 2) "-START")
>
> I didn't look at the code, but I doubt this is correct. The whole point
> of using `string-match-p' is to not affect the match data, so I would
> expect this change to possibly break the immediately following
> (match-string 1) etc. (depending on the exact regexp of course -- as I
> say I didn't actually check it, but in any case this is not a safe
> change AFAIK). You could try wrapping the `string-match' form inside
> `save-match-data' to prevent that.
Thanks for the review. I was just emulating what Carsten did in a
similar fix in git commit 8dd4baf without much thought of side effects.
I used this change to test a recent post about using org-special-blocks.
I don't normally use org-special-blocks in my setup and this change made
it work for me for the simple test case on the mailing list.
I'll try to post a better patch for this tomorrow.
Regards,
Bernt
- [Orgmode] [PATCH] Emacs 22 support, Bernt Hansen, 2010/06/01
- [Orgmode] Re: [PATCH] Emacs 22 support, Štěpán Němec, 2010/06/01
- [Orgmode] Re: [PATCH] Emacs 22 support,
Bernt Hansen <=
- [Orgmode] [PATCH] Emacs 22 support, Bernt Hansen, 2010/06/02
- Re: [Orgmode] [PATCH] Emacs 22 support, Carsten Dominik, 2010/06/03
- Re: [Orgmode] [PATCH] Emacs 22 support, Bernt Hansen, 2010/06/03
- [Orgmode] [PATCH] Emacs 22 support - use org-string-match-p, Bernt Hansen, 2010/06/03
- [Orgmode] Re: [PATCH] Emacs 22 support - use org-string-match-p, Štěpán Němec, 2010/06/03
- [Orgmode] Re: [PATCH] Emacs 22 support - use org-string-match-p, Bernt Hansen, 2010/06/03
- [Orgmode] Re: [PATCH] Emacs 22 support - use org-string-match-p, Štěpán Němec, 2010/06/03
- Re: [Orgmode] Re: [PATCH] Emacs 22 support - use org-string-match-p, Carsten Dominik, 2010/06/04
- Re: [Orgmode] [PATCH] Emacs 22 support - use org-string-match-p, Carsten Dominik, 2010/06/04