bug#21737: Acknowledgement (24.5; Minor feature request: `comint-previous-matching-input-from-input' and point)

[Stephen Perry]

> On 17 Nov 2015, at 00:58, Juri Linkov <juri@linkov.net> wrote:
> 
>> +(defcustom comint-move-point-to-end-on-match nil
>> +  "Controls whether point moves to the end of the matched input.
>> +If nil, then point remains in the current position.
>> +If t, point moves to the end of matched input."
>> +  :type 'boolean
>> +  :group 'comint)
>> +
>> (defvaralias 'comint-scroll-to-bottom-on-output 
>> 'comint-move-point-for-output)
>> 
>> (defcustom comint-scroll-show-maximum-output t
>> @@ -1220,7 +1227,8 @@ comint-previous-matching-input-from-input
>>     (comint-previous-matching-input
>>      (concat "^" (regexp-quote comint-matching-input-from-input-string))
>>      n)
>> -    (goto-char opoint)))
>> +    (unless comint-move-point-to-end-on-match
>> +      (goto-char opoint))))
> 
> I wonder could we use the logic of 'track-eol' here?  This means that going
> to the previous command lines starting at the end of the command line will
> keep to the ends of command lines.

It’s a trivial change to make, but I’m not sure if having it do
the same thing will be annoying for people who don’t want
track-eol turned on everywhere?