Replace <var> with man page markup
Change sub-headings from 'command level' to 'command context'
Details
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Looks great, apart from a few nits below.
usr.sbin/ctld/ctl.conf.5 | ||
---|---|---|
71–72 | "Context" name is much better than "level", but it's still kind of inconsistent - few lines below it says "Opens a target configuration section". Perhaps we should call it "Global section" instead, or change "section" to "context" everywhere? | |
72 | Perhaps the whole "The following statements are available at ..." line could be dropped, since it's redundant? Same for similar lines for other contexts. | |
240–241 | The "/ prefixlen" part is optional; can it be reflected somehow? Same for the other initiator-portal occurrence below. |
Added a lot more markup, and made it more consistent
Improved the sentence flow
Normalized some of the text.
Change the order of some definitions from:
- Description
- what happens if you don't define it
- otherwise, this happens (if it is defined)
to the more logical flow of:
- Description
- What happens when it is defined
- What happens when it is not defined
I make it a point not to disagree with trasz when he is right and has approved a man page. ;-)