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)
Fri, Dec 20, 3:05 PM
Unknown Object (File)
Wed, Dec 11, 8:51 AM
Unknown Object (File)
Mon, Dec 9, 6:27 AM
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)
Nov 24 2024, 11:22 AM
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 Skipped
Unit
Tests Skipped

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

Remember to dump the date.

421

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

445

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

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
usr.bin/
top/
110 lines

Diff 48608

View Options

usr.bin/top/top.1

Loading...