| Age | Commit message (Collapse) | Author |
|---|
|
making the code simpler and easier to understand.
No functional change.
|
|
to the start of the next column correctly.
Fixing a problem found by jmc@ in sysctl(3), reminded by kettenis@.
|
|
and render it just like normal text.
Minimal fix of a formatting bug in operator(7) reported by ray@.
|
|
skip output functions, but not *_endparse;
problem reported by kristaps@
|
|
fix NULL vs 0, etc. No behaviour change.
|
|
We now have sufficient practical experience to know what we want,
so this is intended to be final:
- provide -Wlevel (warning, error or fatal) to select what you care about
- provide -Wstop to stop after parsing a file with warnings you care about
- provide consistent exit status codes for those warnings you care about
- fully document what warnings, errors and fatal errors mean
- remove all other cruft from the user interface, less is more:
- remove all -f knobs along with the whole -f option
- remove the old -Werror because calling warnings "fatal" is silly
- always finish parsing each file, unless fatal errors prevent that
This commit also includes a couple of related simplifications behind
the scenes regarding error handling.
Feedback and OK kristaps@; Joerg Sonnenberger (NetBSD) and
Sascha Wildner (DragonFly BSD) agree with the general direction.
|
|
directly instead - calling libevent functions after fork() w/o
event_reinit() is a bad idea, even if in this case it was harmless.
|
|
already dup()d again later. Fixes a leak seen by espie@.
|
|
used as filename
|
|
as suggested by Thomas de Grivel <billitch at gmail dot com> on tech@.
While here, remove the SEE ALSO entries (but not the cross references
at the appropriate places!) to fnmatch(3), re_format(7), and pr(1).
Their relevance is restricted to one single option each, the latter
is mostly obsolete nowadays anyway, and the number of SEE ALSO entries
is so large that SEE ALSO is starting to look confusing.
"yes" jmc@
|
|
It is always defined in the preamble using .ds when used in manuals.
Since we now support .ds, it is no longer necessary to provide it.
Triggered by a bug report from Thomas Jeunet, patch by kristaps@.
|
|
just in the same way as \s (font size) escapes, and handle \s escapes
not having an explicit plus or minus sign.
This fixes the very ugly rendering of "C++" in gcc(1).
Reported by Thomas Jeunet, fixed by kristaps@.
|
|
Issue reported by Aldis Berzoja, fix by kristaps@.
|
|
from kristaps@.
|
|
Don't use MAX, it doesn't exist in the default namespace on Solaris.
From Joerg Sonnenberger, with a critical fix by kristaps@.
|
|
in man_pmacro() and mdoc_pmacro(). In particular, no need to use
isgraph(3) here, that has already been done in main.c.
Joint work by Kristaps and myself, ok kristaps@.
|
|
because isgraph(3) returns true for some eight-bit characters.
ok kristaps@
|
|
Coverity CID 1660: Plug memory leaks.
Coverity CID 2989: Use the error path to return.
Coverity CID 3212: Don't return, goto out in all cases so that cleanup happens.
ok otto@
|
|
|
|
for, so use that. ok djm
|
|
|
|
reopen stdin/stdout to /dev/null when forking for ControlPersist.
prevents tools that fork and run a captive ssh for communication from
failing to exit when the ssh completes while they wait for these fds to
close. The inherited fds may persist arbitrarily long if a background
mux master has been started by ControlPersist. cvs and scp were effected
by this.
"please commit" markus@
|
|
some other style tweaks
|
|
help and ok millert@ (ok deraadt@ for an earlier version of this diff)
|
|
Honnef, patch from Thomas Adam.
|
|
mode when already in a different mode. Reported by "Florian".
|
|
|
|
removing it accidentally.
|
|
when the client tty backs up too much, just stop updating the tty and
only update the internal screen. Then when the tty recovers, force a
redraw.
This prevents a dodgy client from causing other clients to go into
backoff while still allowing tmux to be responsive (locally) when seeing
lots of output.
|
|
Honnef.
|
|
|
|
"multiprocessor" in a consistent way.
written with lots of good advice from jmc@, who noted that changing
hier(7) was not enough and carefully reviewed this diff.
ok jmc@
|
|
danielm at bluetiger dot net, PR 6234.
OK millert
|
|
at bluetiger dot net, PR 6234.
OK millert
|
|
have multiple items, and .Pa for path names;
|
|
|
|
|
|
of by a tab; so allow the tab in mandoc, too.
Bug found by me, fix by kristaps@, "sure" deraadt@.
|
|
Patch from kristaps@, analoguous to mdoc_term.c rev. 1.100.
With permission from deraadt@ to still fix bugs in mandoc.
|
|
data channels (i.e. TunnelForward):
Calculate local_consumed correctly in channel_handle_wfd() by measuring
the delta to buffer_len(c->output) from when we start to when we finish.
The proximal problem here is that the output_filter we use in portable
modified the length of the dequeued datagram (to futz with the headers
for !OpenBSD).
In channel_output_poll(), don't enqueue datagrams that won't fit in the
peer's advertised packet size (highly unlikely to ever occur) or which
won't fit in the peer's remaining window (more likely).
In channel_input_data(), account for the 4-byte string header in
datagram packets that we accept from the peer and enqueue in c->output.
report, analysis and testing 2/3 cases from wierbows AT us.ibm.com;
"looks good" markus@
|
|
! com>, thanks!
OK deraadt
|
|
with strange devices and (b) since there appears to be a bull in the
poll code in libevent as well...
requested by nicm who is away
|
|
input line, not after the first element. Since free-form text lines
only have a single element in literal mode, this is only relevant for
macro lines inside literal displays, and only for those containing
more than one macro. Fixes e.g. awk(1) and boot_config(8).
Note this fix differs from what kristaps@ committed to bsd.lv because
those changes introduce regressions with respect to blank lines in
literal mode.
With permission from deraadt@ to still fix bugs in mandoc.
|
|
"%option noinput" is specified. OK deraadt@
|
|
|
|
|
|
version replaces some strlc{py,at} verbosity with xasprintf() at
the request of markus@
|
|
"looks ok" markus@
|
|
appear in lexical order and make our ssh-keygen comply. ok markus@
|
|
addresses combinations. It hasn't worked for ages and we have supported
the more commen "[addr]:port" format for a long time. ok jmc@ markus@
|
