coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests: document CMP/PRE/POST in unit test module


From: Assaf Gordon
Subject: Re: [PATCH] tests: document CMP/PRE/POST in unit test module
Date: Tue, 02 Apr 2013 12:11:43 -0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.4) Gecko/20120510 Icedove/10.0.4

Thanks for the quick reply.
Here's a better patch.

Bernhard Voelker wrote, On 04/02/2013 04:03 AM:

> s/PRE,POST/PRE, POST
> due to the line length it may be worth adding a line break.

Done. Also added IN_PIPE .

> Close square brackets, and move blank character to after the comma:

Done.

> 2 notes:
> * According to the code, instead of a plain string, 'data' can also be a HASH.
> * If the file name is '@AUX@', then it is replaced.

I do not fully understand those uses, so I can't really explain them.
When are these useful?

> Furthermore, IN, AUX, and EXIT also do not seem to be documented yet.
> Do you like to document these, too?
> 

I've added "IN" and "IN_PIPE".
"EXIT" was already mentioned.
"AUX" - I do not know what it does...

-gordon



Attachment: 0001-tests-document-more-test-keys-in-unit-test-module.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]