Re: [PATCH 1/2] metadata: add tests grouping support

Cyril Hrubis <[email protected]>
Newsgroups gmane.linux.ltp
Message-ID <[email protected]>
Hi!
> I was thinking that having `@group foo` is cleaner. When we start to have
> multiple groups it can be really to handle multiple groups: for instance,
> we might end up having one line over 80 chars in the description.
> 
> @groups foo1 bar1 foo2 bar2 foo3 bar3 ..
> 
> Instead, we might have
> 
> @group foo1
> @group bar1
> @group foo2
> @group bar2
> @group foo3
> @group bar3
> 
> These strings are short, but if we have long names it can be weird
> to read having all of them on a single line.

I wasn't saying that we should support only one @groups command, rather
than that we would do:


@groups foo1 foo2 foo3
@groups bar1 bar2 bar3

That would allow us to write reasonably short lines and limit the number
of lines to a reasonable number as well.

-- 
Cyril Hrubis
[email protected]

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.