Re: git: 0050289464fa - main - style(9): white space after ; and around binary operators

From: Mark Johnston <markj_at_freebsd.org>
Date: Thu, 16 Oct 2025 23:58:34 UTC
On Thu, Oct 16, 2025 at 04:21:45PM -0700, Rick Macklem wrote:
> Please be aware of the following snippets from "man 9 style":
> 
> In general code can be considered "new code" when it makes up about 50%
> or more of the file(s) involved. This is enough to break precedents in
> the existing code and use the current style guidelines.
> 
> Stylistic changes (including whitespace changes) are hard on the source
> repository and are to be avoided without good reason.

+1

This kind of change accomplishes nothing except to be a source of merge
conflicts for others to deal with.

> rick
> 
> On Thu, Oct 16, 2025 at 3:05 PM David E. O'Brien <obrien@freebsd.org> wrote:
> >
> > CAUTION: This email originated from outside of the University of Guelph. Do not click links or open attachments unless you recognize the sender and know the content is safe. If in doubt, forward suspicious emails to IThelp@uoguelph.ca.
> >
> > The branch main has been updated by obrien:
> >
> > URL: https://cgit.FreeBSD.org/src/commit/?id=0050289464fa56ec79e060b8d8378b9ff7145a0e
> >
> > commit 0050289464fa56ec79e060b8d8378b9ff7145a0e
> > Author:     David E. O'Brien <obrien@FreeBSD.org>
> > AuthorDate: 2025-10-15 05:22:00 +0000
> > Commit:     David E. O'Brien <obrien@FreeBSD.org>
> > CommitDate: 2025-10-16 22:02:50 +0000
> >
> >     style(9): white space after ; and around binary operators
> >
> >     in for() loops.  Also, use 'while', where only the
> >     conditional test of 'for' was used.
> >
> >     Reviewed by: sjg