Re: master 050b830 3/3: Do interactive mode tagging for finder.el

[Eli Zaretskii]

> @@ -423,7 +423,7 @@ FILE should be in a form suitable for passing to 
> `locate-library'."
>  
>  (defun finder-mouse-select (event)
>    "Select item in a Finder buffer with the mouse."
> -  (interactive "e")
> +  (interactive "e" finder-mode)
>    (with-current-buffer (window-buffer (posn-window (event-start event)))
>      (goto-char (posn-point (event-start event)))
>      (finder-select)))

I think this part is dangerous: mouse commands are not necessarily
issued with the buffer on which they operate being current, so
conditioning such commands on the mode of the current buffer is
ill-advised.  Even if currently this marking doesn't produce any
visible problems (does it?), it could do so in the future, if and when
we decide to base more on this tagging than we do now.

I think this part should be reverted, and in the future commands bound
to mouse clicks should not be tagged with any major mode.

Thanks.