Details
- Reviewers
emaste jilles tmunro - Group Reviewers
manpages - Commits
- rS348359: Add posixshmcontrol(1) page.
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 24541
Event Timeline
| usr.bin/posixshmcontrol/posixshmcontrol.1 | ||
|---|---|---|
| 37 | Shouldn't POSIX be capitalized here, since it's an abbreviation? E.g. re_format(7) spells it correctly. Ditto the one below. | |
| 66 | Native speakers may need to correct me here, but IMHO it should be plural contents in this case. | |
| 72 | ... displayed by the subcommand. | |
| 120 | Native speaker's advice needed, but probably To show the contents ... here. | |
| usr.bin/posixshmcontrol/posixshmcontrol.1 | ||
|---|---|---|
| 37 | Yes. | |
| 42 | Where does this come from? AFAICT most man pages just use literal ... and a few \&... | |
| 66 | I would say: It allows inspecting existing segments, dumping their metadata or contents, and unlinking them. However I think uncountable noun (content) works too, the content of the existing segments. | |
| 69 | the system (or the file system) | |
| 74–76 | This sentence is a little confusing, maybe break into two? | |
Ok, LGTM; I'm not an expert on man page markup but can be adjusted after commit if needed.