[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: address@hidden: redisplay abort if GLYPH_DEBUG turned on]
From: |
Kim F. Storm |
Subject: |
Re: address@hidden: redisplay abort if GLYPH_DEBUG turned on] |
Date: |
Fri, 07 Jan 2005 14:17:29 +0100 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/21.3.50 (gnu/linux) |
address@hidden (Kim F. Storm) writes:
> Miles Bader <address@hidden> writes:
>
>> address@hidden (Kim F. Storm) writes:
>>> There are other places which set stop_charpos to 0, so
>>> there is IT_CHARPOS <= stop_charpos is not an invariant.
>>>
>>> So Miles' change to the xassert is ok. But the comment need to be modified
>>> as well.
>>
>> The thing that makes me nervous is that there seem to also be other
>> similar xasserts, e.g. [xdisp.c:2727]:
>>
>> xassert (STRINGP (it->string)
>> || (it->stop_charpos >= BEGV
>> && it->stop_charpos >= IT_CHARPOS (*it)));
>>
>> and [xdisp.c:4068]:
>>
>> xassert (it->stop_charpos >= BEGV
>> && it->stop_charpos <= it->end_charpos);
>>
>> implying that there _are_ some sorts of rules for stop_charpos, but I
>> have no idea what they are. [I haven't had any crashes involving these
>> other xasserts.]
>
> You may be right -- perhaps there is a different way to fix the original
> problem... I will look into it.
I just installed a different fix for the original problem.
So stop_charpos is no longer set to 0 outside strings.
--
Kim F. Storm <address@hidden> http://www.cua.dk