Page MenuHomeFreeBSD

Fix posixshmcontrol nits.
ClosedPublic

Authored by markj on Sep 9 2020, 3:41 PM.
Tags
None
Referenced Files
F167141658: D26376.id76816.diff
Wed, Aug 19, 10:50 AM
F167115365: D26376.id76816.diff
Wed, Aug 19, 5:32 AM
F167112579: D26376.id77049.diff
Wed, Aug 19, 4:56 AM
Unknown Object (File)
Tue, Aug 18, 8:07 PM
Unknown Object (File)
Tue, Aug 18, 10:13 AM
Unknown Object (File)
Sat, Aug 15, 5:42 PM
Unknown Object (File)
Wed, Aug 12, 4:49 PM
Unknown Object (File)
Wed, Aug 12, 4:48 PM
Subscribers

Details

Reviewers
kib
Group Reviewers
manpages
Commits
rS365749: Fix some posixshmcontrol nits.
Summary
  • Exit with an error if no path is specified.
  • Man page typo.
  • Error message typo.

Diff Detail

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

Event Timeline

markj requested review of this revision.Sep 9 2020, 3:41 PM
markj created this revision.

I do not object against returning error if subcommand is not specified, or if no path is given, but why ?

This revision is now accepted and ready to land.Sep 9 2020, 3:53 PM
In D26376#586472, @kib wrote:

I do not object against returning error if subcommand is not specified, or if no path is given, but why ?

The usage message implies that at least one path must be given. I think it is also more conventional to return an error instead of silently doing nothing, and it helps catch bugs (e.g., someone writes "posixshmcontrol create $foo" in a shell script but forgets to assign to foo). Indeed, I noticed because in some manual testing I made a similar mistake.

This revision was automatically updated to reflect the committed changes.