Page MenuHomeFreeBSD

recv.2: Fix a typo and remove an unneeded Tn macro
ClosedPublic

Authored by 0mp on Apr 26 2018, 12:50 AM.
Referenced Files
Unknown Object (File)
Dec 23 2023, 10:59 AM
Unknown Object (File)
Dec 13 2023, 11:41 PM
Unknown Object (File)
Nov 20 2023, 12:56 PM
Unknown Object (File)
Apr 8 2023, 10:23 PM
Unknown Object (File)
Mar 3 2023, 4:33 PM
Unknown Object (File)
Feb 1 2023, 5:30 PM

Details

Summary

mandoc -Tlint recv.2 suggested to remove the Tn macro.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kevans added a subscriber: kevans.
kevans added inline comments.
libc/sys/recv.2
299 ↗(On Diff #41874)

I would poke manpages about this part- spacing in man pages always squigs me out, but IIRC this isn't one of those "really needs a space there" scenarios.

bjk added inline comments.
libc/sys/recv.2
299 ↗(On Diff #41874)

No space is needed; it only comes into play on lines that interpret macros (i.e., start with '.').

libc/sys/recv.2
299 ↗(On Diff #41874)

Ah, ok, I wasn't sure if there were other things like the comma after e.g. =)

This revision is now accepted and ready to land.May 20 2018, 1:37 PM

The reason the lint tool flagged up the Tn macro, mdoc(7) states:
"Tn
Supported only for compatibility, do not use this in new manuals. Even
though the macro name (``tradename'') suggests a semantic function,
historic usage is inconsistent, mostly using it as a presentation-level
macro to request a small caps font."

eadler added inline comments.
libc/sys/recv.2
299 ↗(On Diff #41874)

igor is a really nice tool to find style issues.

libc/sys/recv.2
299 ↗(On Diff #41874)

Mm, nice. I forgot about igor. Thanks!

This revision was automatically updated to reflect the committed changes.