Problems remaining in FOR-RELEASE

Problems remaining in FOR-RELEASE

[Richard Stallman]

Would people please turn their attention to debugging the problems
listed in FOR-RELEASE?

Re: Problems remaining in FOR-RELEASE

[Kim F. Storm]

Richard Stallman <rms@gnu.org> writes:

> Would people please turn their attention to debugging the problems
> listed in FOR-RELEASE?

+-----------------------------------------------------
| ** Investigate face cache related crash.
|  
| Indications: Face cache seems to have been cleared, but
| redisplay uses an invalidated face_id with FACE_FROM_ID
| which then returns a NULL pointer.  Said to happen with
| isearch faces.


I installed fixes related to this problem recently, and
have not received any indications that the problem still
exists.  So if some of you still see the problem, please
tell me.

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

Re: Problems remaining in FOR-RELEASE

[Johan Bockgård]

storm@cua.dk (Kim F. Storm) writes:

> +-----------------------------------------------------
> | ** Investigate face cache related crash.
> |  
> | Indications: Face cache seems to have been cleared, but
> | redisplay uses an invalidated face_id with FACE_FROM_ID
> | which then returns a NULL pointer.  Said to happen with
> | isearch faces.
>
>
> I installed fixes related to this problem recently, and have not
> received any indications that the problem still exists. So if some
> of you still see the problem, please tell me.

I can still make it crash:

----------------------------------------------------------------

Using `C-x 3' in `emacs -nw -Q' after evaluating the following code
makes emacs crash:

(set-display-table-slot
 standard-display-table
 'vertical-border
 (let* ((face 'isearch)
        (face-offset (lsh (face-id face) 19)))
   (+ face-offset ?\|)))

----

(gdb) p face
$1 = (struct face *) 0x0
(gdb) bt
#0  turn_on_face (f=0x903710, face_id=27) at term.c:1911
#1  0x0000000000467d26 in write_glyphs (string=0x2a96c7f628, len=24)
    at term.c:965
#2  0x0000000000411db2 in update_frame_line (f=0x903710, vpos=0)
    at dispnew.c:5656
#3  0x0000000000410fab in update_frame_1 (f=0x903710, force_p=0, 
    inhibit_id_p=0) at dispnew.c:5154
#4  0x000000000040f297 in update_frame (f=0x903710, force_p=0, 
    inhibit_hairy_id_p=0) at dispnew.c:3853
#5  0x0000000000429484 in redisplay_internal (preserve_echo_area=9451280)
    at xdisp.c:10432
[...]

----------------------------------------------------------------

-- 
Johan Bockgård