[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Links in WoMan buffer should not call `man'
From: |
Chong Yidong |
Subject: |
Re: Links in WoMan buffer should not call `man' |
Date: |
Tue, 05 Sep 2006 16:56:11 -0400 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (gnu/linux) |
Michael Welsh Duggan <address@hidden> writes:
>> Some time later, I am realizing that string_match_1 is using
>> current_buffer's case table. Since the string being matched against
>> might have nothing to do with the current buffer, this seems
>> incorrect. Is this a red herring?
>
> More data: The *Message* buffer at dump time has a case_canon_table
> with three extra slots, all nil.
This patch removes the bug, and is probably the correct thing to do.
Any objections to my installing it?
*** emacs/src/regex.c.~1.211.~ 2006-02-21 14:36:47.000000000 -0500
--- emacs/src/regex.c 2006-09-05 16:53:54.000000000 -0400
***************
*** 2162,2168 ****
Lisp_Object eqv_table;
! if (!RE_TRANSLATE_P (translate))
{
EXTEND_RANGE_TABLE (work_area, 2);
work_area->table[work_area->used++] = (start);
--- 2162,2170 ----
Lisp_Object eqv_table;
! if (!RE_TRANSLATE_P (translate)
! || (eqv_table = XCHAR_TABLE (translate)->extras[2],
! NILP (eqv_table)))
{
EXTEND_RANGE_TABLE (work_area, 2);
work_area->table[work_area->used++] = (start);
***************
*** 2170,2177 ****
return -1;
}
- eqv_table = XCHAR_TABLE (translate)->extras[2];
-
for (; start <= end; start++)
{
enum case_type this_type;
--- 2172,2177 ----
- Re: Links in WoMan buffer should not call `man', Juri Linkov, 2006/09/02
- Re: Links in WoMan buffer should not call `man', Richard Stallman, 2006/09/04
- Re: Links in WoMan buffer should not call `man', Juri Linkov, 2006/09/04
- Re: Links in WoMan buffer should not call `man', Michael Welsh Duggan, 2006/09/04
- Re: Links in WoMan buffer should not call `man', Michael Welsh Duggan, 2006/09/04
- Re: Links in WoMan buffer should not call `man', Michael Welsh Duggan, 2006/09/05
- Re: Links in WoMan buffer should not call `man', Michael Welsh Duggan, 2006/09/05
- Re: Links in WoMan buffer should not call `man',
Chong Yidong <=
- Re: Links in WoMan buffer should not call `man', Richard Stallman, 2006/09/06
- Re: Links in WoMan buffer should not call `man', Richard Stallman, 2006/09/07
- Re: Links in WoMan buffer should not call `man', Richard Stallman, 2006/09/06
- Re: Links in WoMan buffer should not call `man', Richard Stallman, 2006/09/06