fix large PID overflow their column in top command

[Matthew Chae]

Hi Denys,

Ok. Good to hear.
If I understand correctly, will you first review my patch and then give me Morris's account to commit the change?
Then, let me send a patch soon.

Br-Matthew
________________________________
From: Denys Vlasenko <vda.linux at googlemail.com>
Sent: Wednesday, November 8, 2023 12:35 PM
To: Matthew Chae <Matthew.Chae at axis.com>
Cc: busybox at busybox.net <busybox at busybox.net>; Christopher Wong <Christopher.Wong at axis.com>
Subject: Re: fix large PID overflow their column in top command

On Wed, Nov 1, 2023 at 10:18 AM Matthew Chae <Matthew.Chae at axis.com> wrote:
>
> Hi maintainer,
>
> I'd like to hear your opinion regarding large PID can overflow its column in top command.
>
> The max PID value can have 7 digits as the 4194304.
> Busybox v1.36 released a new patch to handle large PID value in top command.
> However, it still causes misalignment between the title and the number for 7 digits or 6 digits PID.
> The presence of a large number of PID and PPID digits in the column may cause overflow, making it impossible to display the entire data accurately.
> Plus,  the large PID causes that user name can't be displayed up to 8 characters and result in the truncation of less than 8 characters.
>
> I can contribute new patch which allocates appropriate space for the number of digits in the PID and PPID to represent the values in each column correctly.
> This can make alignment between the title and the number for 7 digits PID and 5 digits PID and also display user name up to 8 characters.
>
> What do you think? If you agree, I will send a patch.

Absolutely, please send a patch. Would be interesting to see a good solution.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/busybox/attachments/20231108/3b1451a9/attachment-0001.html>