From: Quentin Schulz <[email protected]> Currently, if NET is disabled, the next feature to be printed will start with a comma and a space which is not pretty. Add the comma and whitespace only when a previous feature has already been shown.
Signed-off-by: Quentin Schulz <[email protected]> --- cmd/ufetch.c | 29 ++++++++++++++++++++--------- 1 file changed, 20 insertions(+), 9 deletions(-) diff --git a/cmd/ufetch.c b/cmd/ufetch.c index bc5db08eee1..900b126b55b 100644 --- a/cmd/ufetch.c +++ b/cmd/ufetch.c @@ -157,26 +157,37 @@ static int do_ufetch(struct cmd_tbl *cmdtp, int flag, int argc, printf(" (%d baud)", gd->baudrate); putc('\n'); break; - case FEATURES: + case FEATURES: { + bool sep = false; + printf("Features:" RESET " "); - if (IS_ENABLED(CONFIG_NET)) - printf("Net"); - if (IS_ENABLED(CONFIG_EFI_LOADER)) - printf(", EFI"); - if (IS_ENABLED(CONFIG_CMD_CAT)) - printf(", cat :3"); + if (IS_ENABLED(CONFIG_NET)) { + printf("%sNet", sep ? ", " : ""); + sep = true; + } + if (IS_ENABLED(CONFIG_EFI_LOADER)) { + printf("%sEFI", sep ? ", " : ""); + sep = true; + } + if (IS_ENABLED(CONFIG_CMD_CAT)) { + printf("%scat :3", sep ? ", " : ""); + sep = true; + } #ifdef CONFIG_ARM64 switch (current_el()) { case 2: - printf(", VMs"); + printf("%sVMs", sep ? ", " : ""); + sep = true; break; case 3: - printf(", full control!"); + printf("%sfull control!", sep ? ", " : ""); + sep = true; break; } #endif printf("\n"); break; + } case RELOCATION: if (gd->flags & GD_FLG_SKIP_RELOC) printf("Relocated:" RESET " no\n"); -- 2.54.0
