userspace: remove vestigial '?' cases from top-level getopt(3) loops

getopt(3) returns '?' when it encounters a flag not present in the in
the optstring or if a flag is missing its option argument. We can
handle this case with the "default" failure case with no loss of
legibility. Hence, remove all the redundant "case '?':" lines.

Prompted by dlg@. With help from dlg@ and millert@.

Link: https://marc.info/?l=openbsd-tech&m=167011979726449&w=2

ok naddy@ millert@ dlg@

Convert quad_t to int64_t and %q to %ll
Convert bzero() to memset() and bcopy() to memcpy()

ok natano@ millert@

pledge to only use "stdio rpath"; ok doug

Put fts_close() where missing.

Not bugs in short-lived commands that call exit() -> _exit() immediately,
but for idempotency.

Originally found in ls(1) by Valgrind. Changes for other commands are
from deraadt@. Reviewed by me, tested in snapshots.

OK deraadt@

fix a regression that i caused in rev. 1.26: *do* report the size of
files listed on the command line, no matter what options are given;
issue reported by edd@; ok millert@ edd@

rearrange -d, again, after giving ingo some dodgy advice;
ok ingo

in usage() and SYNOPSIS, sort options as suggested by jmc@

Add the -d flag (limit display depth) for compatibility with FreeBSD,
DragonFly, NetBSD and GNU coreutils, even though it's not POSIX.
Actually, this simplifies the code rather than complicating it.

Because -a and -d need not be mutually exclusive (as observed by
millert@) and -s is identical to -d 0, -a and -s are no longer
mutually exclusive, but -as and -sa are now the same as -s.
That is explicitly allowed by POSIX.

Based on a patch from William Orr <will at worrbase dot com>,
but extensively massaged and HISTORY added by me.

feedback and ok millert@

Use errc/warnc to simplify code.
Also, in 'ftp', always put the error message last, after the hostname/ipaddr.

ok jsing@ krw@ millert@

a few things can be static to avoid need for protos

cast one blocksize occurrence to unsigned long, fixes du(1)'s output
for directories larger than one terabyte; while here, wrap a long line.

problem found by Chano Antuna and diff written with lots of good
advice from millert@, thanks!

ok millert@

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

Use FTS_ROOTLEVEL instead of 0 when checking fts_level.

If both -h and -k are specified, the latter takes precedence.
Prompted by jmc@, OK otto@

lint says Pflag isn't needed. i concur

sort options + sync usage();

Actually commit right (r&b tree) version, instead of hash table
one. Previous commit from wrong tree. Spotted by Andre Lucas <andre
at ae-35 dot com> ok millert@ canacar@

- use fmt_scaled(3) instead of home grown function to print -h numbers
- do not use a linear list to keep track of inodes with link count > 2,
use a red & black tree. Based on freebsd code that uses auto-sizing
hash maps; this tree version by canacar@.

ok millert@ canacar@

Don't want text in err() call following malloc failures.

ok henning@

protos

mostly ansi cleanup; pval ok

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999. Proofed by myself and Theo.

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

fix 0B case; morten@hotpost.dk

first pass at a -Wall cleanup

-h overrides BLOCKSIZE; matt@anzen.com

Add -h option ("human-readable") as df. Based on suggestions and code from
Matt Bing <matt@anzen.com>.

correct return value; from FreeBSD

-r for xpg conformance

getopt(3) returns -1 when out of args, not EOF, whee!

Implement a -c (Grand Total) option, a'la GNU du(1).
From Luke Mewburn <lukem@telstra.com.au>, NetBSD PR #2805.
Also -Wall happiness.

rcsid

branches: 1.1.1;
Initial revision

Convert quad_t to int64_t and %q to %ll
Convert bzero() to memset() and bcopy() to memcpy()

ok natano@ millert@

pledge to only use "stdio rpath"; ok doug

Put fts_close() where missing.

Not bugs in short-lived commands that call exit() -> _exit() immediately,
but for idempotency.

Originally found in ls(1) by Valgrind. Changes for other commands are
from deraadt@. Reviewed by me, tested in snapshots.

OK deraadt@

fix a regression that i caused in rev. 1.26: *do* report the size of
files listed on the command line, no matter what options are given;
issue reported by edd@; ok millert@ edd@

rearrange -d, again, after giving ingo some dodgy advice;
ok ingo

in usage() and SYNOPSIS, sort options as suggested by jmc@

Add the -d flag (limit display depth) for compatibility with FreeBSD,
DragonFly, NetBSD and GNU coreutils, even though it's not POSIX.
Actually, this simplifies the code rather than complicating it.

Because -a and -d need not be mutually exclusive (as observed by
millert@) and -s is identical to -d 0, -a and -s are no longer
mutually exclusive, but -as and -sa are now the same as -s.
That is explicitly allowed by POSIX.

Based on a patch from William Orr <will at worrbase dot com>,
but extensively massaged and HISTORY added by me.

feedback and ok millert@

Use errc/warnc to simplify code.
Also, in 'ftp', always put the error message last, after the hostname/ipaddr.

ok jsing@ krw@ millert@

a few things can be static to avoid need for protos

cast one blocksize occurrence to unsigned long, fixes du(1)'s output
for directories larger than one terabyte; while here, wrap a long line.

problem found by Chano Antuna and diff written with lots of good
advice from millert@, thanks!

ok millert@

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

Use FTS_ROOTLEVEL instead of 0 when checking fts_level.

If both -h and -k are specified, the latter takes precedence.
Prompted by jmc@, OK otto@

lint says Pflag isn't needed. i concur

sort options + sync usage();

Actually commit right (r&b tree) version, instead of hash table
one. Previous commit from wrong tree. Spotted by Andre Lucas <andre
at ae-35 dot com> ok millert@ canacar@

- use fmt_scaled(3) instead of home grown function to print -h numbers
- do not use a linear list to keep track of inodes with link count > 2,
use a red & black tree. Based on freebsd code that uses auto-sizing
hash maps; this tree version by canacar@.

ok millert@ canacar@

Don't want text in err() call following malloc failures.

ok henning@

protos

mostly ansi cleanup; pval ok

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999. Proofed by myself and Theo.

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

fix 0B case; morten@hotpost.dk

first pass at a -Wall cleanup

-h overrides BLOCKSIZE; matt@anzen.com

Add -h option ("human-readable") as df. Based on suggestions and code from
Matt Bing <matt@anzen.com>.

correct return value; from FreeBSD

-r for xpg conformance

getopt(3) returns -1 when out of args, not EOF, whee!

Implement a -c (Grand Total) option, a'la GNU du(1).
From Luke Mewburn <lukem@telstra.com.au>, NetBSD PR #2805.
Also -Wall happiness.

rcsid

branches: 1.1.1;
Initial revision