correct indentation; no functional change
ok tb@

Delete obsolete /* ARGSUSED */ lint comments.

ok miod@ millert@

Before calling setrlimit(), pull cur up to max.
ok millert

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.

show more ...

blkfree() takes no action for NULL pointer so callers can avoid checking.

ok jca@

No need to declare variable name as static in dounsetenv(). Also, drop a
redundant call to free() since name is always freed upon return.

From Michael W. Bombardieri, with some tweaks by myself.

Passing NULL to free() is fine; from Michael W. Bombardieri.

Fix pasting of long (>BUFSIZ) lines in csh with filec enabled. NUL-terminating
the input buffer instructs csh that the buffer contains a complete command. This
is wrong and should only happen when bu

Fix pasting of long (>BUFSIZ) lines in csh with filec enabled. NUL-terminating
the input buffer instructs csh that the buffer contains a complete command. This
is wrong and should only happen when buffer is not full, otherwise more data has
to be read in order form a complete command.

While here, do not print the prompt again when the input exceeds the input
buffer and while inserting a line continuation (backslash).

show more ...

Postpone printing of prompt if filec is enabled in csh. Any I/O should be
performed first when canonical mode has been disabled on the tty just like ksh
does. Discovered by the regress tests.

Joint

Postpone printing of prompt if filec is enabled in csh. Any I/O should be
performed first when canonical mode has been disabled on the tty just like ksh
does. Discovered by the regress tests.

Joint work with bluhm@, ok deraadt@

show more ...

Don't reinvent rlim_t; when printing it use %llu and cast to
unsigned long long

ok natano@ tedu@

Replace handrolled xfree() function by directly using free(3)

OK mmcc@

Clean up cruft made visible by the NLS unifdefing:
* remove setlocale() calls
* remove write-only variable AsciiOnly
* remove now unused string constants STRLANG, STRLC_CTYPE
* remove hardcoded suppo

Clean up cruft made visible by the NLS unifdefing:
* remove setlocale() calls
* remove write-only variable AsciiOnly
* remove now unused string constants STRLANG, STRLC_CTYPE
* remove hardcoded support for ISO8859-1

ok millert@

show more ...

unifdef BUILTIN (unused), NLS (always set), SHORT_STRINGS (always set)
and remove a few unused defines; no binary changes; ok tedu@ zhuk@

lint is dead: delete the trivial uses of /* VARARGS[0-9]+ */
(others require more care)

delete tons more casts

delete lots of unnecessary ptr_t casts, and drop a little NULL in as well

Use reallocarray() and remove a few archaic memory allocation practices.
ok deraadt

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
(th

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

show more ...

print vmemoryuse immediately after vmemoryuse in limit output

vmemsize -> vmemoryuse for consistency with tcsh and document. mickey@ OK

conditional use of RLIMIT_VMEM (like ksh)

print the whole hint for bad limit modifiers. ok otto pedro

protos

kill printf builtin. no reason to be a builtin, but furthermore, the
handling of format strings is quite non-compliant.

de-register and ansify functions