bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#886: 23.0.60; Info-mouse-follow-nearest-node fails


From: Stephen Berman
Subject: bug#886: 23.0.60; Info-mouse-follow-nearest-node fails
Date: Mon, 15 Sep 2008 15:48:24 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux)

On Mon, 15 Sep 2008 14:11:38 +0200 martin rudalics <rudalics@gmx.at> wrote:

>> Does the following patch fix it?
>>
>> *** emacs/lisp/info.el.~1.544.~      2008-09-11 11:49:36.000000000 +0200
>> --- emacs/lisp/info.el       2008-09-15 11:22:53.000000000 +0200
>> ***************
>> *** 3123,3129 ****
>>          (Info-goto-node
>>           (Info-extract-menu-item (match-string-no-properties 1)) fork)
>>          t)))
>> !       (error "Point neither on reference nor in menu item description")))
>>
>>   ;; Common subroutine.
>>   (defun Info-try-follow-nearest-node (&optional fork)
>> --- 3123,3130 ----
>>          (Info-goto-node
>>           (Info-extract-menu-item (match-string-no-properties 1)) fork)
>>          t)))
>> !       (unless (eq this-command 'Info-mouse-follow-nearest-node)
>> !    (error "Point neither on reference nor in menu item description"))))
>>
>>   ;; Common subroutine.
>>   (defun Info-try-follow-nearest-node (&optional fork)
>
> Yes - the following seems simpler:
>
> !       (eq this-command 'Info-mouse-follow-nearest-node)
> !     (error "Point neither on reference nor in menu item description")))

Oops, yes of course.

> I'm not sure though whether we should raise an error for mouse-2 clicks.
> But I also fail to understand why we should raise an error at all.

I agree an error here is unnecessary and it would be better to remove
it.  I doubt any existing code depends on it.

Steve Berman






reply via email to

[Prev in Thread] Current Thread [Next in Thread]