stdcxx-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Martin Sebor <se...@roguewave.com>
Subject Re: [PATCH] stdcxx target warning counting
Date Fri, 13 Oct 2006 01:25:45 GMT
Andrew Black wrote:
> Take 3 attached, altered some, slightly altered log at end.

Thanks. Committed as follows:
http://svn.apache.org/viewvc?view=rev&rev=463535

A few issues/suggestions:

First, the width of output now exceeds 80 characters per line.
That's okay in batch mode but we need to do something about that
in interactive mode. Maybe we could drop the PERCNT column? Or
rename it to PCNT and shrink its width? Or integrate it into
the STATUS column (display the percentage instead of the exit
status only when the latter is zero). We could also reduce the
width of the times columns.

Another option might be to set the width of the first column
to the length of the longest target. That would be a bit more
involved (we'd have to scan all command line arguments before
processing them and compute the width that way) but it would
reduce the overall width quite a bit in most cases.

Second, I think we might want to move the logs to a separate
directory (kind of like the .d files are in .depend) so as to
avoid the clutter in the tests and examples directories. And
we should either add a dependency of the run target on the
logs and/or indicate the absence of the .log file(s) somehow.

Martin

Mime
View raw message