Re: [LTP] [PATCH v3 1/2] testcases/lib/test.sh: Added ROD_SILENT() and ROD()

[Cyril Hrubis <chrubis@suse.cz>]

Hi!
> > I update the repository and got the documentation. It looks good, and
> > thank you very much.
> > 
> Maybe you can write the descriptive file such as tar/00_Descriptions.txt,
> but I don't know if it's necessary.

My opinion is that the less documentation is needed the better. Which
means that one should write clean code and if possible avoid cryptic
constructs and comment only parts that needs to be.

In LTP we usually add a short description of the testcase in a comment
at the start of the test source. Looking at the tar testcase, I would be
better to move the description to the comment at the start of the
tar_tests.sh. Having it in separate file is not good idea.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list