bug#23179: 25.0.92; Restore `M-,' to continue etags search

[Dmitry Gutov]

On 04/05/2016 06:12 PM, Eli Zaretskii wrote:

> I hope you are joking.  Since when does anyone need to disclaim
> responsibility for some code?  And even if someone does, "git blame"
> will blame the guilty parties right away.

I mostly mean feature requests, bugs-which-are-really-missing-features, 
and so on.

> Copying a function only to change a line or two sounds extreme to me.

The idea is that the other UI should really be a new 
xref-show-xrefs-function, that's what that variable is there for (you 
can make it a defcustom). It would be better to avoid coupling the two UIs.

> The point is to be able to tell people who, like Anders, don't want to
> see the UI to use the option to get what they want, instead of arguing
> with them trying to convince them that the UI is for their best.

I'm just wondering if the new looks were a minor part of his complaint, 
and not seeing the first match quickly, a more significant one. Anyway, 
I don't mind.

Here's a more technical concern: if you revisit the discussion 
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=20489, we've mentioned that 
next-error-find-buffer, like it's currently written, really wants the 
next-error-last-buffer to be visible. Otherwise, it's prone to switch to 
using next-error-function from some other, visible buffer.