Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Feb 2023 04:58:15 GMT
From:      Kyle Evans <kevans@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 6431b5ab6bba - stable/12 - top: fix sorting by pid (-o pid)
Message-ID:  <202302160458.31G4wFWm057838@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by kevans:

URL: https://cgit.FreeBSD.org/src/commit/?id=6431b5ab6bba2dbf7a22c00e6a3ce2afd1bde436

commit 6431b5ab6bba2dbf7a22c00e6a3ce2afd1bde436
Author:     Kyle Evans <kevans@FreeBSD.org>
AuthorDate: 2023-02-13 06:39:04 +0000
Commit:     Kyle Evans <kevans@FreeBSD.org>
CommitDate: 2023-02-16 04:57:54 +0000

    top: fix sorting by pid (-o pid)
    
    The 'pid' order was added, but the equivalent comparator was not back in
    8b800d3af2d8c ("Add the PID column to the list of sort keys.").
    `ordernames` indices need to match `compares` indices in order for this
    to work.
    
    A future commit will restructure all of this to make it less error prone
    going forward (see D37083).
    
    MFC after:      3 days
    Reviewed by:    pstef (D37083)
    
    (cherry picked from commit deb63adf945d446ed91a9d84124c71f15ae571d1)
---
 usr.bin/top/machine.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/usr.bin/top/machine.c b/usr.bin/top/machine.c
index 6116a06bb28b..d41b3e4f6b74 100644
--- a/usr.bin/top/machine.c
+++ b/usr.bin/top/machine.c
@@ -1552,6 +1552,7 @@ int (*compares[])(const void *arg1, const void *arg2) = {
 	compare_ivcsw,
 	compare_jid,
 	compare_swap,
+	compare_pid,
 	NULL
 };
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202302160458.31G4wFWm057838>