[LTP] 回复: [PATCH v2] cleanup: Unify the annotation style and remove Redundant code

Petr Vorel pvorel@suse.cz
Wed Mar 17 12:34:34 CET 2021


Hi all,

> > Hi Gongyi,

> > > Change 'DESCRIPTION' to 'Description' and change 'ALGORITHM' to 
> > > 'Algorithm', and remove Redundant tst_reap_children since 
> > > tst_reap_children has been called in the test skeleton.

> > Reviewed-by: Petr Vorel <pvorel@suse.cz>

> Hi Petr,

> If we want to unify the format of 'Description' and 'Algorithm' in test, it seems redundant to do the conversion in testinfo.pl, right?

> Can we use either of them(not both of them) because it is unnecessary to do the same thing twice.

I'd keep both: this is for cleanup of old approach, conversion in testinfo.pl in
case there is inconsistency in the future.

But not a strong opinion about it, it's not that important.

Kind regards,
Petr


More information about the ltp mailing list