Subject: Re: bug#77775: 30.1; Proced performance

From: Roland Winkler <[email-masked]>
Date: Mon, 14 Apr 2025 05:38:26 -0500

On Mon, Apr 14 2025, Azeem Hasan wrote:
> Looking at how the process-alist passed to `proced-format' is
> constructed, I think a better change that preserves current behavior
> would be:
>
> -  (let ((standard-attributes
> -         (car (proced-process-attributes (list-system-processes))))
> +  (let ((standard-attributes (car process-alist))

That's my point:

The code used to be

  (let ((standard-attributes
         (car (proced-process-attributes (list (emacs-pid)))))

and that's what the comment refers to that is still present in the code.
Somehow, this got replaced by

 (let ((standard-attributes
        (car (proced-process-attributes (list-system-processes))))

and I want to find out what this change was supposed to accomplish.

Your patch

  (let ((standard-attributes (car process-alist))

uses somewhat randomly the first process of process-alist.