> From: Robert Weiner <rsw@gnu.org>
> Date: Wed, 27 Sep 2017 12:01:50 -0400
>
> The doc for posn-window is incomplete. posn-set-point does not handle drag
> events whose end point argument is a frame, rather than a window.
> This patch fixes both of these.
> ! `posn-window': The window or frame of the event end.
I think we should say a bit more about this. For example:
`posn-window': The window of the event end, or its frame if event
end point belongs to no window.
> (defun posn-set-point (position)
> "Move point to POSITION.
> Select the corresponding window as well."
> ! (if (not (windowp (posn-window position)))
> ! (error "Position not in text area of window"))
> ! (select-window (posn-window position))
> (if (numberp (posn-point position))
> (goto-char (posn-point position))))
>
> --- 1170,1182 ----
> (defun posn-set-point (position)
> "Move point to POSITION.
> Select the corresponding window as well."
> ! (if (framep (posn-window position))
> ! (progn (if (not (windowp (frame-selected-window (posn-window
> position))))
> ! (error "Position not in text area of window"))
> ! (select-window (frame-selected-window (posn-window position))))
Why should we select the selected-window on that frame in this case?
Can youdescribe a use case where this would be a useful behavior?
In any case, the change in posn-set-point's behavior, if we agree on
it, should be described in NEWS. The changes also lack a log entry.
I'm okay with installing the documentation changes in the release
branch, but the change in posn-set-point should be discussed first, as
I'm not sure we want that. If we agree on making that change, it
should go to master, I think.