Thanks for all the feedback, I have a few queries.

Thanks. Please be sure to test the new faces with the following Emacs
configurations:

. GUI frames with dark background
. GUI frames with light background
. TTY frames with dark and light backgrounds and with:
- 8 colors
- 16 colors

Do you know if there's an easy way I can test the faces on 8/16 colour terminals?

(proced-run-status-code, proced-executable. proced-memory-default)
(proced-memory-mb, proced-pgrp): New faces.

Minor, but do you mean to add a closing paren at the end of each line? I saw this format in the commit log:

(proced-run-status-code, proced-executable. proced-memory-default,

proced-memory-mb, proced-pgrp): New faces.

This should probably be specified as percentage of total memory. Or maybe
there should be a separate defcustom for the percentage, and the condition
should use both. Just a single absolute threshold seems to cover only some
reasons for highlighting processes with large memory footprint.

I was thinking highlighting based on percentage memory would be more suited to the "mem" process attribute (granted though this is not implemented). Though in hindsight a global threshold may not make sense if you're connecting to remote systems with varying amounts of RAM. Perhaps the two thresholds could mark a percentage, say 10% and 50% of total memory?

I haven't tried your patch, but I'm curious whether it works also for
proced buffers of remote systems.

Hi Michael, I did a quick check yesterday and it seemed to work as expected.

This should happen only for proced running on the local system.

Thanks, this should be fixed in the most recent patch.

I've attached a more up to date patch, with changes to faces (mainly adding better defaults for light backgrounds), and a NEWS entry. Also attached are images showing the current colour schemes on light backgrounds and terminals.

Thanks, Laurence