bug#28864: 25.3.50; next-error-no-select does select

[Dmitry Gutov]

On 10/29/17 12:07 AM, Juri Linkov wrote:

> I believe this patch is a change for the better:

Could you resend it as unified diff? Or push to a branch. 
diff-context->unified doesn't seem to work on the patch you sent.

Thanks!

> 1. makes next-error-function buffer-local

Interesting.

> 2. sets both buffer-local and global values

Why? If next-error-find-buffer-function can find the appropriate buffer, 
it can use the local value in it.

> 3. adds customizable next-error-find-buffer-function
>
> 4. moves window-on-frame-visibility code to separate function
>     that can be used to customize for backward-compatibility

Do we get a built-in alternative to this value? If we just provide a 
customization point, that nice, but not a significant improvement.

> 5. adds next-error-select-buffer to manually switch to another
>     next-error capable buffer

Not having tried the patch yet... Will the user have to do that after 
semi-accidentally opening a changelog-mode buffer, in order not to 
switch to its error list?

> 6. message to show which next-error buffer is currently used

Every time we call `next-error'?