Page MenuHomeFreeBSD
Differential D17369

top(1): rework DESCRIPTION OF MEMORY section
ClosedPublic
Actions

Authored by 0mp on Oct 1 2018, 10:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 3:37 PM
Unknown Object (File)
Tue, Nov 26, 2:56 PM
Unknown Object (File)
Tue, Nov 26, 11:54 AM
Unknown Object (File)
Tue, Nov 26, 11:52 AM
Unknown Object (File)
Sun, Nov 24, 11:22 AM
Unknown Object (File)
Sat, Nov 23, 3:52 PM
Unknown Object (File)
Sat, Nov 23, 12:05 PM
Unknown Object (File)
Thu, Nov 21, 5:12 PM
View All Files
Subscribers
0mp
imp
krion
mat

Details

Reviewers
eadler
bcr
yuripv
mat
krion
Group Reviewers
manpages
Restricted Owners Package(Owns No Changed Paths)
Commits
rS339080: top(1): Rework DESCRIPTION OF MEMORY section.
Summary

Due to markup issues, the DESCRIPTION OF MEMORY section is rather unreadable; rework it a bit, using subsections for different lines of the top output, and move it closer to description.

While here, pet manlint ordering other sections as expected.

Diff Detail

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

Event Timeline

yuripv created this revision.Oct 1 2018, 10:47 AM
Herald added a reviewer: manpages. · View Herald TranscriptOct 1 2018, 10:48 AM
Herald added a subscriber: imp. · View Herald Transcript
0mp accepted this revision as: 0mp.Oct 1 2018, 11:02 AM
0mp added a subscriber: 0mp.

mdoc changes seem reasonable.

Is there a reason why we don't want to sort those stats alphabetically?

usr.bin/top/top.1
2 ↗(On Diff #48605)

Remember to dump the date.

421 ↗(On Diff #48605)

I think this might look better with -width "Uncompressed"

445 ↗(On Diff #48605)

Unnecessary newline?

This revision is now accepted and ready to land.Oct 1 2018, 11:02 AM
yuripv marked 4 inline comments as done.Oct 1 2018, 11:10 AM
yuripv added inline comments.
usr.bin/top/top.1
421 ↗(On Diff #48605)

Yes, had the same thought, but then forgot about it.

yuripv updated this revision to Diff 48608.Oct 1 2018, 11:10 AM
yuripv marked an inline comment as done.
This revision now requires review to proceed.Oct 1 2018, 11:10 AM
yuripv added a comment.Oct 1 2018, 11:12 AM

The order currently follows what we have in the actual top output, and seems logically more correct than sorting alphabetically.

0mp accepted this revision as: 0mp.Oct 1 2018, 11:12 AM
0mp added a reviewer: bcr.

Looks good to me.

This revision is now accepted and ready to land.Oct 1 2018, 11:12 AM
eadler accepted this revision.EditedOct 1 2018, 3:06 PM

Thanks!

0mp commandeered this revision.Oct 1 2018, 3:12 PM
0mp edited reviewers, added: yuripv; removed: 0mp.
0mp added reviewers: mat, krion.
0mp added subscribers: krion, mat.

Cool. I can get it committed.

@krion, @mat, may I have your approval?

krion accepted this revision.Oct 1 2018, 3:14 PM

I think from now we agreed with mat@ you get blank approval from us for doc/ and src/ changes assuming these changes are approved from doc/ or src/ committers.

0mp added a comment.Oct 1 2018, 3:20 PM
In D17369#370505, @krion wrote:

I think from now we agreed with mat@ you get blank approval from us for doc/ and src/ changes assuming these changes are approved from doc/ or src/ committers.

Thank you, @krion & @mat. That's a great honor. Thank you for your trust.

When it comes to the revision itself, I'll now submit it to re@.

0mp added a comment.Oct 2 2018, 8:14 AM

Committed! Thanks, @yuripv_yuripv.net!

Closed by commit rS339080: top(1): Rework DESCRIPTION OF MEMORY section. (authored by 0mp). · Explain WhyOct 2 2018, 8:14 AM
This revision was automatically updated to reflect the committed changes.
Owners added a reviewer: Restricted Owners Package.Oct 2 2018, 8:14 AM

Revision Contents
Changeset List

PathSize
head/
usr.bin/
top/
110 lines

Diff 48644

View Options

head/usr.bin/top/top.1

Loading...