[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#50256: thing-at-mouse
From: |
Juri Linkov |
Subject: |
bug#50256: thing-at-mouse |
Date: |
Wed, 01 Sep 2021 18:42:35 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (x86_64-pc-linux-gnu) |
>> 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.
- bug#50256: thing-at-mouse, (continued)
- bug#50256: thing-at-mouse, Eli Zaretskii, 2021/09/05
- bug#50256: thing-at-mouse, martin rudalics, 2021/09/06
- bug#50256: thing-at-mouse, Eli Zaretskii, 2021/09/06
- bug#50256: thing-at-mouse, martin rudalics, 2021/09/06
- bug#50256: thing-at-mouse, Eli Zaretskii, 2021/09/06
- bug#50256: thing-at-mouse, Juri Linkov, 2021/09/06
- bug#50256: thing-at-mouse, Juri Linkov, 2021/09/12
- bug#50256: thing-at-mouse, martin rudalics, 2021/09/02
- bug#50256: thing-at-mouse, Juri Linkov, 2021/09/03
- bug#50256: thing-at-mouse, Eli Zaretskii, 2021/09/02
bug#50256: thing-at-mouse,
Juri Linkov <=
bug#50256: thing-at-mouse, Eli Zaretskii, 2021/09/01
bug#50256: thing-at-mouse, Juri Linkov, 2021/09/12
bug#50256: thing-at-mouse, Juri Linkov, 2021/09/12