Page MenuHomeFreeBSD

usr.bin/procstat: Add missing usage and getopt(3) options
ClosedPublic

Authored by kdrakehp_zoho.com on May 25 2017, 9:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 10:52 AM
Unknown Object (File)
Feb 2 2024, 7:46 AM
Unknown Object (File)
Feb 1 2024, 12:26 PM
Unknown Object (File)
Jan 28 2024, 6:22 AM
Unknown Object (File)
Dec 20 2023, 4:59 AM
Unknown Object (File)
Dec 12 2023, 10:07 AM
Unknown Object (File)
Nov 27 2023, 11:25 PM
Unknown Object (File)
Oct 12 2023, 1:13 AM
Subscribers

Details

Summary
  • Add the missing option 'n' to the getopt(3) string
  • Add the missing options 'libxo' and 'N' to the usage message
  • Add the missing options 'M' and 'N' to the man-page
Test Plan

The man-page's usage section, the usage message, the getopt(3)
string, and its switch statement were checked against eachother.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

usr.bin/procstat/procstat.c
57 ↗(On Diff #28820)

At the risk of making usage longer, I think keeping [-a | pid | core ...] on it's own line is clearer.

168 ↗(On Diff #28820)

'L' should come before 'l'. style(9) says the case statements should also be sorted, but I know that will create conflicts with D10916 so let's fix that after landing that change.

  • Move the last arguments of usage to a separate line
  • Correct the order of 'L' and 'l' in the getopt string

Looks good to me. I'll commit shortly.

This revision is now accepted and ready to land.May 26 2017, 7:50 PM
This revision was automatically updated to reflect the committed changes.