svn commit: r299588 - head/usr.sbin/binmiscctl

[Bruce Evans]

On Fri, 13 May 2016, Don Lewis wrote:

> Log:
>  Revert r299584:
>    Mark usage() as __dead2 so that Coverity doesn't think that execution
>    continues after the call and uses a negative array subscript.
>
>  Requested by:	bde

Thanks.

What was the problem that confused Coverity?

This reminds me that even compilers can see that usage() doesn't return,
by looking ahead and even inlining usage().  This breaks debugging and
profiling.  clang doesn't support the -funit-at-a-time and
-fno-inline-functions-called-once needed to reduce its excessive inlining.

Bruce