Re: problem report #99

["Davis Herring" <herring@lanl.gov>]

> 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.