Details
- Reviewers
emaste jilles tmunro - Group Reviewers
manpages - Commits
- rS348359: Add posixshmcontrol(1) page.
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| usr.bin/posixshmcontrol/posixshmcontrol.1 | ||
|---|---|---|
| 36 ↗ | (On Diff #57960) | Shouldn't POSIX be capitalized here, since it's an abbreviation? E.g. re_format(7) spells it correctly. Ditto the one below. |
| 65 ↗ | (On Diff #57960) | Native speakers may need to correct me here, but IMHO it should be plural contents in this case. |
| 71 ↗ | (On Diff #57960) | ... displayed by the subcommand. |
| 119 ↗ | (On Diff #57960) | Native speaker's advice needed, but probably To show the contents ... here. |
| usr.bin/posixshmcontrol/posixshmcontrol.1 | ||
|---|---|---|
| 36 ↗ | (On Diff #57960) | Yes. |
| 41 ↗ | (On Diff #57960) | Where does this come from? AFAICT most man pages just use literal ... and a few \&... |
| 65 ↗ | (On Diff #57960) | 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. |
| 68 ↗ | (On Diff #57960) | the system (or the file system) |
| 73–75 ↗ | (On Diff #57960) | 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.