[PATCH] Enhance tests.

Jim Meyering jim at meyering.net
Mon Oct 4 11:32:19 UTC 2010


Jeff Darcy wrote:
...
> Removed "-v" from iwhd command line.

Actually, please don't remove -v (--verbose)
from the test's iwhd invocation.

I've just encountered one instance where using -v there
made iwhd print a diagnostic that saved me some time.
I was following attributes-related examples from
doc/warehouse_api_final.md and tried to set an attribute
without an operator.  The --verbose output informed me of
the error (I'll update the API doc) and probably saved me
from having to look at the code or use a debugger.

Perhaps that is an argument for converting some DPRINTF diagnostics
(which print only when "verbose" is set) to be unconditional,
If we err on the side of verbosity for now, we'll get more diagnostics
in the t/*.log files, and that's almost always a good thing, especially
in early development.


More information about the iwhd-devel mailing list