Details
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 33052 Build 30435: arc lint + arc unit
Event Timeline
Formatting seems good to me :) I've checked both mdoc and HTML outputs.
sbin/devd/devd.conf.5 | ||
---|---|---|
508 | This is a bug fix, right? |
sbin/devd/devd.conf.5 | ||
---|---|---|
330 | +1, this might further improve navigating around the manual page. |
sbin/devd/devd.conf.5 | ||
---|---|---|
329–333 | The manual list -type first (-column is the type) and -compact last in the synopsis. It gives no admonition that it must be first. A quick google search doesn't turn it up either. And none of the man pages in the source tree do that either. I don't see that either siginfo.3 or sigevent.3 use -compact... I'll keep it last... But I can get 2 more characters from Description if I remove trailing white space in System and Subsystem headings. I can get away w/o having the final column arg, so I'll do that. | |
330 | Sure. Easy enough. |
sbin/devd/devd.conf.5 | ||
---|---|---|
508 | Yes. Added a comment to the commit message. |
sbin/devd/devd.conf.5 | ||
---|---|---|
329–333 | I was referring to this part of the mdoc(7) manual: Bl -type [-width val] [-offset val] [-compact] [col ...] Begin a list. Lists consist of items specified using the It macro, containing a head or a body or both. Clearly, the optional list of columns [col ...] comes after the optional -compact. I was worried that If the parser was like getopt(3) it would assuming all tokens after the first one that doesn't start with - are column widths, thus treating -compact as a column width. However, it seems like the parser doesn't work like that since it does seem to render the same both ways. |