svn commit: r365071 - in head/sys: net net/altq net/route net80211 netgraph netgraph/atm netgraph/atm/ccatm netgraph/atm/sscfu netgraph/atm/sscop netgraph/atm/uni netgraph/bluetooth/common netgraph...

[Kevin Bowling]

I disagree that the problem is intractable.  It's just a decision and
it has a one time cost with long term benefits like paying off a high
interest loan.  The intractability opinion seemed justifiable for a
long time but it's been proven false by other communities,
particularly Go and Rust and there is nothing syntactically special
about these languages that enable this; it's just a decision to make
the style fit an extant formatter.  An arbitrary formater may leave a
little bit of annoyance to each person's taste, but that is a tiny
drop in the bucket compared to never having to discuss and especially
correct (which may /seem/ helpful but is pretty offputting to
newcomers).  A tool does it, and it takes the wind out of any passive
aggressive bike shed opportunities from either maintainer or
contributor.  It sucks that downstreams have to fall in line, but that
doesn't stop progress on any other major changes in FreeBSD.

On Fri, Sep 4, 2020 at 7:57 PM Warner Losh <imp at bsdimp.com> wrote:
>
> On Fri, Sep 4, 2020, 7:05 PM Mark Linimon <linimon at lonesome.com> wrote:
>
> > On Fri, Sep 04, 2020 at 02:15:04PM -0400, Andrew Gallatin wrote:
> > > and I also anticipate it will cause problems with MFCs
> >
> > And existing PRs and DRs.
> >
>
> Or we could just not bother we these changes at all. It's a pipe dream we
> will ever be style(9) compliant in all our code, or that we can magically
> have a tool to enforce in new commits. We have better things to worry
> about. We should continue to ignore this non problem and for new users
> point them at the 95% correct format thing to run their submitted patches
> if they submit something too far out of whack.
>
> The last sweep deleted a boatload of blank lines that were in there on
> purpose. Not worth adding them back, but still annoying to no real benefit.
>
> I just don't see the benefits at all of doing anything here. The few
> reviews that I've seen mention it seem to be the right level of effort.
>
> Warner
>
> >
> _______________________________________________
> svn-src-head at freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-head
> To unsubscribe, send any mail to "svn-src-head-unsubscribe at freebsd.org"