GNU bug report logs - #50256
thing-at-mouse

Previous Next

Package: emacs;

Reported by: Juri Linkov <juri <at> linkov.net>

Date: Sun, 29 Aug 2021 17:44:02 UTC

Severity: normal

Tags: fixed

Fixed in version 28.0.50

Done: Juri Linkov <juri <at> linkov.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Juri Linkov <juri <at> linkov.net>
To: martin rudalics <rudalics <at> gmx.at>
Cc: 50256 <at> debbugs.gnu.org, Lars Ingebrigtsen <larsi <at> gnus.org>
Subject: bug#50256: thing-at-mouse
Date: Wed, 01 Sep 2021 18:42:35 +0300
>> So maybe this fix should be implemented in Fpos_visible_in_window_p,
>> i.e. at the end of this code
>>
>>    if (EQ (pos, Qt))
>>      posint = -1;
>>    else if (!NILP (pos))
>>      posint = fix_position (pos);
>>    else if (w == XWINDOW (selected_window))
>>      posint = PT;
>>    else
>>      posint = marker_position (w->pointm);
>
> Using the position of point of the current buffer when WINDOW is
> specified doesn't make any sense.  Moreover the Elisp manual says that
>
>   "The argument POSITION defaults to the current position of point in
>   WINDOW"
>
> which doesn't make sense if point is taken from a buffer not shown in
> WINDOW.  So the fix should be implemented in Fpos_visible_in_window_p.

Actually, WINDOW is not specified, but defaults to the selected window.
But still you point is valid: using the position of point of the current buffer
for the selected window doesn't make sense.

>> it should get position from the selected window's buffer?
>
> I don't understand this last sentence though.

I meant to set posint to (window-point), but written in C.




This bug report was last modified 3 years and 315 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.