Date: Wed, 7 May 2025 14:14:40 -0400 From: John Baldwin <jhb@FreeBSD.org> To: Warner Losh <imp@bsdimp.com>, Olivier Certner <olce@freebsd.org> Cc: "Piotr P. Stefaniak" <pstef@freebsd.org>, src-committers <src-committers@freebsd.org>, dev-commits-src-all@freebsd.org, dev-commits-src-branches@freebsd.org Subject: Re: git: 154ccb0196cd - stable/14 - ps(1): Update some options' conformance/practice comments Message-ID: <175576a0-78fc-4d5f-a3f4-f940d083dec8@FreeBSD.org> In-Reply-To: <CANCZdfrBtR2WSkv8zD1BjBAi-bqhdXwy2Do82Kt8Gb9OL7ZmMw@mail.gmail.com> References: <202505011951.541JpiQV068751@gitrepo.freebsd.org> <aBXN1oednpVU4yav@freefall.freebsd.org> <2988597.slGk94SIus@ravel> <CANCZdfrBtR2WSkv8zD1BjBAi-bqhdXwy2Do82Kt8Gb9OL7ZmMw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 5/5/25 09:29, Warner Losh wrote: > On Mon, May 5, 2025, 6:13 AM Olivier Certner <olce@freebsd.org> wrote: > >>>> - /*- >>>> + /* >>> >>> No opinion on which it should be, but the minus character is meaningful >>> within the context of indent(1) - it won't reformat "block" comments. >> >> Thanks for pointing that out, I didn't know. I think removing them was >> OK, as these block comments are really nothing special. Actually, I >> probably should have reformatted them to eliminate the margin for all but >> the first line. >> > > Almost nobody uses indent these days on a regular basis. We can drop it for > that reason alone, even if the comments were formatted by hand... we likely > need to start tagging those with clang format tags.. Eh, the clang format tags are ugly for this use case. If using CommentPragmas doesn't already handle this case, I'd rather add a new option to clang-format to disable reflowing comments when this is present. We already have this widely used throughout the tree, and it would better to fix the tool than gratuitous churn. We probably clear clang-format's bar of being a project of a sufficient size that they will accept patches from us for new formatting options. -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?175576a0-78fc-4d5f-a3f4-f940d083dec8>