[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#19129: 25.0.50; recenter-positions is missing a reference to move-to
From: |
H. Dieter Wilhelm |
Subject: |
bug#19129: 25.0.50; recenter-positions is missing a reference to move-to-window-line-top-bottom |
Date: |
Fri, 21 Nov 2014 00:28:42 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
Drew Adams <drew.adams@oracle.com> writes:
>> in the documentation for recenter-positions was forgotten that it is
>> also used in move-to-window-line-top-bottom. I added below patch as
>> an illustration.
>
> Actually no. The doc for a function should rarely, if ever, speak
> about users (callers) of that function. It should simply say what
> the function does, and perhaps what it can be used for if it is
> intended for users.
>
> But it should typically not try to list all of its callers, or even
> the most important callers.
>
> If this is no longer just a helper for `recenter-top-bottom'
> then perhaps reference to even that function can be removed
> from the `recenter-positions' doc.
I agree, my impulse went exactly into the opposite and wrong direction
:-/
Hope you'll remove the reference from recenter-positions and please
close the bug (or can I do this as well?)
Thanks for your explanations
Dieter
--
Best wishes
H. Dieter Wilhelm
Darmstadt, Germany
bug#19129: 25.0.50; recenter-positions is missing a reference to move-to-window-line-top-bottom, Eli Zaretskii, 2014/11/21