Re: problem report #99

[Davis Herring]

> 4265        /* Note that this uses F to determine which terminal to look
> at.
> 4266           If there is no valid info, it does not store anything
> 4267           so x remains nil.  */
> 4268        x = Qnil;
> 4269
> 4270        /* XXX Can f or mouse_position_hook be NULL here? */

This comment says it all.  When this is skipped:

> At conditional (1): "f != 0" taking true path
> At conditional (2): "((f)->terminal)->mouse_position_hook != 0" taking
> false path
>
> 4271        if (f && FRAME_TERMINAL (f)->mouse_position_hook)
> 4272          (*FRAME_TERMINAL (f)->mouse_position_hook) (&f, 0,
> &bar_window,
> 4273                                                      &part, &x, &y,
> &time);

...then this can't happen:

> At conditional (8): "x != Qnil" taking true path
> At conditional (9): "obj == Qnil" taking true path
>
> 4296        if (!NILP (x) && NILP (obj))
>
> Event uninit_use_in_call: Using uninitialized value "bar_window" in call
> to function "make_lispy_movement" [model]
> Also see events: [var_decl]
>
> 4297          obj = make_lispy_movement (f, bar_window, part, x, y, time);
> 4298      }

No bug.

Davis

-- 
This product is sold by volume, not by mass.  If it appears too dense or
too sparse, it is because mass-energy conversion has occurred during
shipping.