Differential D53827

beep: Sort usage and man page options
Needs ReviewPublic
Actions

Authored by jlduran on Wed, Nov 19, 4:02 PM.

Details

Reviewers

pauamma_gundo.com

Group Reviewers

manpages

Summary

Sort usage and man page options, mention possible minimum and maximum
values, and fix punctuation marks.

PR: 291092

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 68743
Build 65626: arc lint + arc unit

Event Timeline

jlduran created this revision.Wed, Nov 19, 4:02 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptWed, Nov 19, 4:02 PM

jlduran requested review of this revision.Wed, Nov 19, 4:02 PM

Harbormaster completed remote builds in B68742: Diff 166780.Wed, Nov 19, 4:02 PM

jlduran added a reviewer: manpages.Wed, Nov 19, 4:04 PM

Manual page changes look OK to me with this nit fixed.

usr.bin/beep/beep.1
57	Consistent naming with usage().

This revision is now accepted and ready to land.Wed, Nov 19, 4:59 PM

Address suggestions from both Phabricator and Bugzilla.

This revision now requires review to proceed.Wed, Nov 19, 5:55 PM

Harbormaster completed remote builds in B68743: Diff 166788.Wed, Nov 19, 5:55 PM

Thank you!
If there are no further suggestions, I'll commit over the weekend.

Revision Contents
Changeset List

Path

Size

usr.bin/

beep/

beep.1

53 lines

beep.c

19 lines

Diff 166788

View Options

usr.bin/beep/beep.1