bug#38561: 27.0.50; Feature request: expose buffer size setting for process filters

[yyoncho <yyoncho@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1232 bytes --]

Hi Eli,

Thank you for taking care of this!

I can confirm that changing the new variable value is yielding the same
results.

I have a question on how am I supposed to use the new setting. Is it
possible to set it per process?

I tried

(with-current-buffer (process-buffer proc)
   (setq-local read-process-output-max (* 1024 1024)))

but it does not work. If it is not possible to use it per process then I
think that I have 2 options:

1. Document it in lsp-mode's readme
2. Override it with a setting in lsp-mode with a bigger value.

Both have clear advantages and disadvantages. Can you advise?

Thanks,
Ivan

On Sat, Dec 21, 2019 at 10:49 AM Eli Zaretskii <eliz@gnu.org> wrote:

> > From: yyoncho <yyoncho@gmail.com>
> > Date: Tue, 10 Dec 2019 19:25:17 +0200
> >
> > Adding some performance numbers from lsp-mode project.
> >
> > (with-current-buffer "DemoApplication.java"
> >   (benchmark-run 10 (lsp-request "workspace/symbol" (list :query "S"))))
> >
> > ;; Emacs from master
> > => (4.203047857 0 0.0)
> >
> > ;; tested with emacs compiled with 1mb buffer.
> > => (3.132925793 0 0.0)
>
> Your wish has been granted, see the new variable
> read-process-output-max.
>
> I'd appreciate testing this with your use cases.
>

[-- Attachment #2: Type: text/html, Size: 1984 bytes --]