Re: Info bug

[David Kastrup]

Juri Linkov <address@hidden> writes:

>> Due to a recent change to Info, I see the following bug:
>>
>> emacs -Q
>> M-x info RET -> Point is on the last info link, instead of point-min.
>>
>> Could anyone who changed Info in the last few days please double-check
>> your work and see if you introduced this bug?
>
> This is caused by the following change in `Info-complete-menu-item':
>
> @@ -2271,8 +2262,7 @@
>    ;; Note that `Info-complete-menu-buffer' could be current already,
>    ;; so we want to save point.
> -  (save-excursion
> -    (set-buffer Info-complete-menu-buffer)
> +  (with-current-buffer Info-complete-menu-buffer
>      (let ((completion-ignore-case t)
>         (case-fold-search t)
>         (orignode Info-current-node)
>
> In theory, these constructs should be equivalent, but they are not.

Read the comment above the change.  It describes the rationale for the
original code and does not jibe with the new code.

You are probably thinking of save-current-buffer rather than
save-excursion.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum