[LTP] [PATCH v2 1/2] lib: Add support for test tags
Petr Vorel
pvorel@suse.cz
Tue Oct 22 16:19:46 CEST 2019
Hi Cyril,
> + fprintf(stderr, "Options\n");
> + fprintf(stderr, "-------\n\n");
I still would remove this extra new line:
fprintf(stderr, "-------\n");
> +
> for (i = 0; i < ARRAY_SIZE(options); i++)
> fprintf(stderr, "%s\n", options[i].help);
> @@ -424,6 +430,28 @@ static void print_help(void)
> fprintf(stderr, "%s\n", tst_test->options[i].help);
> }
> +static void print_test_tags(void)
> +{
> + unsigned int i;
> + const struct tst_tag *tags = tst_test->tags;
> +
> + printf("\nTags\n");
> + printf("----\n\n");
And here:
printf("----\n");
Otherwise LGTM.
Kind regards,
Petr
More information about the ltp
mailing list