Re: xasserts and vertical motion.

[storm@cua.dk (Kim F. Storm)]

David Kastrup <dak@gnu.org> writes:

> There are now several xasserts in the vertical motion routines that
> are supposed to check for some assumptions about progress made on
> display.  I strongly recommend taking them out.
>
> a) they are causing frequent aborts in normal editing operations.  At
> a time where one has to refer people to the trunk Emacs for quite a
> bit of functionality that feels somewhat unfortunate.

I think some of those xasserts are bogus -- it's just hard to _prove_.

In any case, I have removed two of them from move_it_vertically_backward
that are often triggered without any indication of anything actually
being wrong (emacs doesn't loop and the display looks fine).

> Those xasserts that replace visual cues with an aborting Emacs do not
> just make using Emacs for serious work harder, but they also actively
> _hinder_ debugging possibly occuring display problems.

I agree.

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk