Page MenuHomeFreeBSD

sockstat: Ensure that there is always a space between columns
ClosedPublic

Authored by markj on Nov 29 2024, 3:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 18, 9:20 PM
Unknown Object (File)
Wed, May 6, 10:56 PM
Unknown Object (File)
Apr 20 2026, 3:16 AM
Unknown Object (File)
Apr 17 2026, 10:49 AM
Unknown Object (File)
Apr 15 2026, 3:06 PM
Unknown Object (File)
Apr 15 2026, 9:21 AM
Unknown Object (File)
Apr 15 2026, 3:18 AM
Unknown Object (File)
Apr 14 2026, 12:05 PM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Nov 29 2024, 3:52 PM
asomers added a subscriber: asomers.

This looks like a good incremental change, although I can't test it right now. But this program is also begging for some TLC. It could benefit from libxo output, and it's also a good candidate for a C++ conversion. The entire -w option seems like an avoidable hack, if output were printed more intelligently. Most ambitiously of all, this program is fairly begging for some kind of "xo_print_table" function, that would automatically size the columns. But your change is still a good one by itself.

This revision is now accepted and ready to land.Nov 29 2024, 6:23 PM

This looks like a good incremental change, although I can't test it right now. But this program is also begging for some TLC. It could benefit from libxo output, and it's also a good candidate for a C++ conversion. The entire -w option seems like an avoidable hack, if output were printed more intelligently. Most ambitiously of all, this program is fairly begging for some kind of "xo_print_table" function, that would automatically size the columns. But your change is still a good one by itself.

I can only agree.