lscpu: simplify cache column output function

Simplify the logic to "always print a ',' for each cache except if it is the
last one.  This is also a preparation patch for printing the cache column for
offline CPUs where it would print one colon too much because of the current
logic.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

diff --git a/sys-utils/lscpu.c b/sys-utils/lscpu.c
index 8326b10815edc8a2bffd810602ad051e8b02e011..607e062f22dd44680f551bc4f165810b77a19370 100644 (file)
--- a/sys-utils/lscpu.c
+++ b/sys-utils/lscpu.c
@@ -823,13 +823,11 @@ print_parsable_cell(struct lscpu_desc *desc, int i, int col, int compatible)
 
                        for (x = 0; x < ca->nsharedmaps; x++) {
                                if (CPU_ISSET_S(i, setsize, ca->sharedmaps[x])) {
-                                       if (j != desc->ncaches - 1)
-                                               putchar(compatible ? ',' : ':');
                                        printf("%d", x);
                                        break;
                                }
                        }
-                       if (x == ca->nsharedmaps)
+                       if (j != 0)
                                putchar(compatible ? ',' : ':');
                }
                break;