[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Assertion failure in set_iterator_to_next

From: Eli Zaretskii
Subject: Re: Assertion failure in set_iterator_to_next
Date: Sun, 11 Apr 2010 06:14:48 +0300

> From: Stefan Monnier <address@hidden>
> Date: Sat, 10 Apr 2010 20:52:45 -0400
> While playing with my nhexl-mode.el I bumped into some assertion failure
> in the redisplay code.  More specifically the assertion
>         xassert (IT_BYTEPOS (*it) == CHAR_TO_BYTE (IT_CHARPOS (*it)));
> at line 6183.
> I think the problem is that the non-bidi code above that line uses
> it->len assuming it holds the length of the char at point (well, at
> IT_CHARPOS(it)), whereas IIUC it holds the length of "the" glyph.

No, it->len should be the length of the multibyte sequence of the
character at IT_CHARPOS (*it).  This is how the basic iteration works,
and if the length is incorrect, we will get garbled display, in
anything but pure 7-bit ASCII buffers.

> In my case it->c holds sometimes the "lf symbol" char (u240a) which comes
> from a display property and it->len is then 3, which is obviously not
> the length of the char at IT_CHARPOS since the buffer I was looking at
> only happens to contain ASCII chars.

I think it->c is not always the character at IT_CHARPOS (*it).  In
particular, this code is under `case GET_FROM_BUFFER', so the iterator
is iterating through the buffer, not through any string.  But as you
say, it->c still holds a character from a string.  I think it->c is
loaded only in get_next_display_element, which is called _after_
set_iterator_to_next advances to the next character.  And if it->what
is anything other than a character, it->c will not be updated at all.

> I use the quick patch below, which seems to fix the problem, but I don't
> know enough of the redisplay to know whether that's the right fix, or
> whether the right fix is to make sure it->len holds some other value.

I don't think it's right.  Can you show a simple test case where this
problem happens?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]