From nobody Sat Jun 28 02:35:18 2025 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4bTc3S4N9Wz60bGH; Sat, 28 Jun 2025 02:35:32 +0000 (UTC) (envelope-from asomers@gmail.com) Received: from mail-ej1-f50.google.com (mail-ej1-f50.google.com [209.85.218.50]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "WR4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4bTc3S27DKz3nty; Sat, 28 Jun 2025 02:35:32 +0000 (UTC) (envelope-from asomers@gmail.com) Authentication-Results: mx1.freebsd.org; none Received: by mail-ej1-f50.google.com with SMTP id a640c23a62f3a-ae36e88a5daso40822866b.1; Fri, 27 Jun 2025 19:35:32 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751078130; x=1751682930; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=lM4O9u8vhRbp5aOoAV4+1PoEo6q97HzX2oXxgTSxT1Q=; b=DXDbLi40CWOiEUYvh0kHMgry31/5YH10KKS7gyWy43xcYYlRZfmNX9+ovq79miYz/r 4xmKw2/VxsurxqP5pHOYLHIk2R835QfPjXrk17PHXCMD5T2dTZixTUi6ErFyB1B6NU+2 lnL2T5v7nwgtAuSx8j4OSJiaqlsLOaEcUGMUQxzt3ixhDOkghGLOeOBgojY7dMm4xxqp YFUFaIGO5MA3hMu/NqxtiRfOdSRXSbpJrjSnr7ya/3kpKumF1tqUF6BDYHgIftwBYkWL //IDexBBZDYRm99aILtTFnOj/vgPzSz+UZsnq2eWClQyMEj7tkAnuKLdmoimNc8Hu16a HfKw== X-Forwarded-Encrypted: i=1; AJvYcCVktDfySmAFatHR0cxh9hSiHnqLhTb9ulYwc8xy+SqkyvdkHJ44KlyhSReoBnq/Euo3N5YFHU7c6RhsPYjHriNN+NV9QrQ=@freebsd.org, AJvYcCWcVBoHcGYWth02RzLwGh0it6oSqZIygup54qAZJ2mrT8MSOD1SfHOnhuJatwgHdOkH9sNc/lr0DVAng6j0Xcobo2XL@freebsd.org X-Gm-Message-State: AOJu0YwfJTtHeLS1jlSxvdO1AZ2SsbLQNkH8XEticqEOXHWPeYXSn228 RENvbdyyhhLHEYGUVhgONXleAas9l6B78fpbRE6ODnyQUlIS+XpnGxl+IxfFeTFhF+NRd6lFiC/ lM6g89fyV85c9xnzsk6OZIdfhRYlPXLR0vA== X-Gm-Gg: ASbGnctShHXef2yejsRUUG6riYRZwD60ni4Rdkp+H1eRAP1t3hX6yrI6g+kKFC6Wg5P OiYUICSnYGSZIBEHcDjaJ/pg1YuDVZCgWFcbAxSHEJtO71gdceW3oKUlqJyTBdI1jVfEyEUT9jS 0VOj3X2rO2vSs6VMc5f0dUFGb3w0KeykRP7A6MuD+qqyk= X-Google-Smtp-Source: AGHT+IGt+Ya+khvXfCF+a3fl66GaTV+MnpKr4Q344Ibmcpbo74p4diDzw8AEVR8LSD/LO/M8EjVShBS6xvmOEc27zCg= X-Received: by 2002:a17:907:3f86:b0:ae0:bd48:b9b9 with SMTP id a640c23a62f3a-ae34fedf159mr528888066b.23.1751078130002; Fri, 27 Jun 2025 19:35:30 -0700 (PDT) List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@FreeBSD.org MIME-Version: 1.0 References: <202506271559.55RFxWdm024724@gitrepo.freebsd.org> In-Reply-To: From: Alan Somers Date: Fri, 27 Jun 2025 20:35:18 -0600 X-Gm-Features: Ac12FXx1d7j9KyIT0Yqtj-4urpp2vLv2Wvbh-r0E50nyfmT4beuyb8bNjIvePfI Message-ID: Subject: Re: git: 0726c6574f88 - main - sockstat: Add automatic column sizing and remove -w option To: Gleb Smirnoff Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org, Damin Rido Content-Type: multipart/alternative; boundary="000000000000e9aebe063898a521" X-Rspamd-Queue-Id: 4bTc3S27DKz3nty X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:15169, ipnet:209.85.128.0/17, country:US] --000000000000e9aebe063898a521 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, Jun 27, 2025 at 7:23=E2=80=AFPM Gleb Smirnoff = wrote: > Hi, > > On Fri, Jun 27, 2025 at 03:59:32PM +0000, Alan Somers wrote: > A> commit 0726c6574f889507e5030173bf4c82c80911394d > A> Author: Damin Rido > A> AuthorDate: 2025-06-12 09:12:49 +0000 > A> Commit: Alan Somers > A> CommitDate: 2025-06-27 15:54:41 +0000 > A> > A> sockstat: Add automatic column sizing and remove -w option > A> > A> Refactor sockstat to dynamically size table columns based on > content. > A> This eliminates the need for the -w option, which is now ignored > A> for backwards compatibility. > A> > A> Numeric columns are now right-aligned for improved readability; > A> previously, they were left-aligned. > A> > A> Unknown fields are now consistently shown as "??" instead of a mix > A> of "", "?", and "?" for output uniformity. > A> > A> Sponsored by: Google, LLC (GSoC 2025) > A> MFC after: 2 weeks > A> Reviewed by: asomers > A> Pull Request: https://github.com/freebsd/freebsd-src/pull/1720 > A> --- > A> usr.bin/sockstat/sockstat.1 | 6 +- > A> usr.bin/sockstat/sockstat.c | 622 > +++++++++++++++++++++++++++----------------- > A> 2 files changed, 392 insertions(+), 236 deletions(-) > > The commit message looks promising, but so far the output of the program > that I > use daily has been made much worsr :( > > First, there is a spurious second newline on every line. Even after the > header > line! How could that pass through minimal testing? > A second newline? Do you mean that the output is double-spaced? Because I do not see that at all. > > Second, the default output now just inserts a huge whitespace "column" in > the > middle of the list, making it extermely wide. Note, that before this > change > the default output was fitting into 80 char terminal. Now it is not > fitting > and without any good reason to do so. > You shouldn't be seeing any column that's pure whitespace. You might be looking at the "PROTO" column, which is often wide because some sockets show "stream (not connected)". The "(not connected)" part isn't new; that's always been there. Does your "whitespace column" go away if you use "-4" or "-6"? If so, Damin do you think we could move "(not connected)" into the "LOCAL ADDRESS" column? It might fit better there. > > I really would prefer this reverted and re-tested by a wider audience > before > pushing it back in. > I'm sorry that your experience has been poor. But I disagree that a full reversion is warranted. > > -- > Gleb Smirnoff > --000000000000e9aebe063898a521 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
On Fri, Jun 27, 2025 at 7:23=E2=80=AFPM Gleb = Smirnoff <glebius@freebsd.org= > wrote:
=C2= =A0 Hi,

On Fri, Jun 27, 2025 at 03:59:32PM +0000, Alan Somers wrote:
A> commit 0726c6574f889507e5030173bf4c82c80911394d
A> Author:=C2=A0 =C2=A0 =C2=A0Damin Rido <rido@FreeBSD.com>
A> AuthorDate: 2025-06-12 09:12:49 +0000
A> Commit:=C2=A0 =C2=A0 =C2=A0Alan Somers <asomers@FreeBSD.org> A> CommitDate: 2025-06-27 15:54:41 +0000
A>
A>=C2=A0 =C2=A0 =C2=A0sockstat: Add automatic column sizing and remove -= w option
A>=C2=A0 =C2=A0 =C2=A0
A>=C2=A0 =C2=A0 =C2=A0Refactor sockstat to dynamically size table column= s based on content.
A>=C2=A0 =C2=A0 =C2=A0This eliminates the need for the -w option, which = is now ignored
A>=C2=A0 =C2=A0 =C2=A0for backwards compatibility.
A>=C2=A0 =C2=A0 =C2=A0
A>=C2=A0 =C2=A0 =C2=A0Numeric columns are now right-aligned for improved= readability;
A>=C2=A0 =C2=A0 =C2=A0previously, they were left-aligned.
A>=C2=A0 =C2=A0 =C2=A0
A>=C2=A0 =C2=A0 =C2=A0Unknown fields are now consistently shown as "= ;??" instead of a mix
A>=C2=A0 =C2=A0 =C2=A0of "", "?", and "?" = for output uniformity.
A>=C2=A0 =C2=A0 =C2=A0
A>=C2=A0 =C2=A0 =C2=A0Sponsored by:=C2=A0 =C2=A0Google, LLC (GSoC 2025)<= br> A>=C2=A0 =C2=A0 =C2=A0MFC after:=C2=A0 =C2=A0 =C2=A0 2 weeks
A>=C2=A0 =C2=A0 =C2=A0Reviewed by:=C2=A0 =C2=A0 asomers
A>=C2=A0 =C2=A0 =C2=A0Pull Request:=C2=A0 =C2=A0h= ttps://github.com/freebsd/freebsd-src/pull/1720
A> ---
A>=C2=A0 usr.bin/sockstat/sockstat.1 |=C2=A0 =C2=A06 +-
A>=C2=A0 usr.bin/sockstat/sockstat.c | 622 +++++++++++++++++++++++++++--= ---------------
A>=C2=A0 2 files changed, 392 insertions(+), 236 deletions(-)

The commit message looks promising, but so far the output of the program th= at I
use daily has been made much worsr :(

First, there is a spurious second newline on every line.=C2=A0 Even after t= he header
line! How could that pass through minimal testing?
A second newline?=C2=A0 Do you mean that the output is double-s= paced?=C2=A0 Because I do not see that at all.
=C2=A0

Second, the default output now just inserts a huge whitespace "column&= quot; in the
middle of the list, making it extermely wide.=C2=A0 Note, that before this = change
the default output was fitting into 80 char terminal.=C2=A0 Now it is not f= itting
and without any good reason to do so.

Y= ou shouldn't be seeing any column that's pure whitespace.=C2=A0 You= might be looking at the "PROTO" column, which is often wide beca= use some sockets show "stream (not connected)".=C2=A0 The "(= not connected)" part isn't new; that's always been there.=C2= =A0 Does your "whitespace column" go away if you use "-4&quo= t; or "-6"?=C2=A0 If so, Damin do you think we could move "(= not connected)" into the "LOCAL ADDRESS" column?=C2=A0 It mi= ght fit better there.
=C2=A0

I really would prefer this reverted and re-tested by a wider audience befor= e
pushing it back in.

I'm sorry that = your experience has been poor.=C2=A0 But I disagree that a full reversion i= s warranted.
=C2=A0

--
Gleb Smirnoff
--000000000000e9aebe063898a521--