Page MenuHomeFreeBSD

domains: rewrite documentation to describe present state
ClosedPublic

Authored by glebius on Sep 10 2022, 8:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 19, 11:00 PM
Unknown Object (File)
Tue, Mar 19, 11:00 PM
Unknown Object (File)
Tue, Mar 19, 11:00 PM
Unknown Object (File)
Feb 3 2024, 7:58 AM
Unknown Object (File)
Jan 4 2024, 8:41 AM
Unknown Object (File)
Dec 24 2023, 8:30 PM
Unknown Object (File)
Dec 23 2023, 2:07 AM
Unknown Object (File)
Nov 30 2023, 12:41 AM

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

debdrup added inline comments.
share/man/man9/domain.9
36

This can be word-smithed, I think.

55

A bit of word-smithing here too, might need to reflow the entire paragraph as a result.

59

Missing particle here.

65

There's a non-specific reference here, and I think the sentence can benefit from being rewritten but without understanding the reference I'm not having much luck coming up with something.

121

Missing particle here, paragraph might need to be reflowed.

216

I'd suggest using existing "X first appeared in N", as that's common across a wide variety of manual pages.

Apply debdrup's suggestions.

Partial review, will resume when brain and keyboard both cooperate.

share/man/man9/domain.9
36

Is that about kernel implementation of sockets for use by userland, or a kernel-internal API?

55

Alternative rewording

59

2 missing articles, not 1. I'm undecided about "an application", "the application", or "applications".11

61–67

What I think this likely means, clarified and copyedited.

65

See my guess below.

68–70

There aren't 3 "protocol" argument, only one (which is 3rd in the argument list)

72

"shall" as a future imperative modal is pretty much extinct.

Add corrections from pauamma

glebius added inline comments.
share/man/man9/domain.9
36

This is KPI (kernel PI) that allows to implement sockets for use by userland. The KPI itself doesn't create any kind of socket, but all existing sockets are built on it.

55

Can you please guys agree on some final wording and I will abide. :)

mhorne added inline comments.
share/man/man9/domain.9
166

.Xr SYSINIT 9

You might add it to SEE ALSO as well/instead.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 14 2022, 9:13 PM
This revision was automatically updated to reflect the committed changes.